musescore / MuseScore

MuseScore is an open source and free music notation software. For support, contribution, bug reports, visit MuseScore.org. Fork and make pull requests!
https://musescore.org
Other
12.06k stars 2.6k forks source link

[MU4 Issue] Many dialog windows have missing app icon in the title bar on Windows #10597

Open DmitryArefiev opened 2 years ago

DmitryArefiev commented 2 years ago

Describe the bug Many dialog windows have missing app icon in title bar on Windows:

MU4 MU4 1 MU4 2

But some dialogs have app icon: MU4 5

Platform information Windows (Win10)

Additional context Doesn't occur in MU3

DmitryArefiev commented 2 years ago

@Tantacrul Maybe to put this to Beta1 but with lower priority? Since it occurs in many windows (on Windows))

Tantacrul commented 2 years ago

As annoying as it is, I can't put it in Beta 1 when there are numerous score / MIDI / audio issues to solve. Let's just wait until all the Muse Hub, Muse Sounds and MuseScore.com upload bugs start coming in!

DmitryArefiev commented 2 years ago

Sure! I understand.

RoeiMeiri9 commented 1 year ago

I wish the title bar would have been custom-made - like the one on the main window where you don't have this screaming bright line that burns my eyes every time I look at it. (By the way - Microsoft made a terrible design choice by turning the title bar white)

cbjeukendrup commented 1 year ago

I wish it could have been custom made! Unless I'm completely missing something, it is so terribly difficult to simply change the color of the window title bar on Windows. I have spent hours in my life just googling how to change the title bar color, but it seems simply not doable in a way that works together with the things that we use (c++, Qt) (without doing crazily complicated stuff that is not worth the effort).

(FWIW, I do think that you can change the title bar color on your own computer. Windows has a sort of dark mode feature, somewhere hidden in Settings.)

RoeiMeiri9 commented 1 year ago

I wish it could have been custom made! Unless I'm completely missing something, it is so terribly difficult to simply change the color of the window title bar on Windows. I have spent hours in my life just googling how to change the title bar color, but it seems simply not doable in a way that works together with the things that we use (c++, Qt) (without doing crazily complicated stuff that is not worth the effort).

(FWIW, I do think that you can change the title bar color on your own computer. Windows has a sort of dark mode feature, somewhere hidden in Settings.)

Maybe this will help: https://stackoverflow.com/questions/30181630/qdialog-remove-title-bar It's not much, but if you can hide the title bar (at least in a dialog), you might be able to add your own custom draggable bar with a header and an X button. I've done this a few times before - but only in WPF and Electron, never in Qt (never programmed in Qt...) It took me a bit of effort, and I believe it's not that bad in Qt either.

Oh and I just checked and my windows settings theme is currently black, and it doesn't affect the title bar at all.

Tantacrul commented 1 year ago

Yeah, I'd opt in future for us completely replacing the popup so we can use our own custom header (a bit like we do with the main title bar).

papoteur-mga commented 1 year ago

Hello, I have a similar behaviour in Linux (Mageia), but on the main window. The icon at top left is an X like generic application. The issue is also for the taskbar in LXQt, but not in Plasma. The icon is well displayed in menu. A friend reported me that the top left is also an X like generic application with the appimage.

papoteur-mga commented 1 year ago

I applied a correction which allows to get the icon in the main window: adding this line:

    app.setWindowIcon(QIcon::fromTheme(QStringLiteral("mscore")));

at line 112: https://github.com/musescore/MuseScore/blob/3969d7975534927ab859c154fcee9a47b0acfe2b/src/appshell/appshell.cpp#L111

cbjeukendrup commented 1 year ago

@papoteur-mga Nice! Would you like to open a Pull Request for this yourself?

papoteur-mga commented 1 year ago

If you can do it, I would be happy. Just a line to add, but I'm not comfortable with PR.

cbjeukendrup commented 1 year ago

@papoteur-mga I tried it out, and unfortunately this doesn't work on Windows... I suspect QIcon::fromTheme is very platform dependent. So we'll need to look further...