MuseScore icon indicating copy to clipboard operation
MuseScore copied to clipboard

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

Open DmitryArefiev opened this issue 3 years ago • 12 comments

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

  • "Do you want to save changes.." dialog
  • New Score
  • Parts
  • Instruments
  • Export
  • Preferences
  • Workspaces
  • Create new workspaces
  • Insert measures
  • About MuseScore
  • About MusicXML
  • Check for update
  • Revert to factory settings
  • All dialogs from Diagnostic tab
  • Migration dialog (import of previous version scores)
  • Error message dialog when trying to add text to a not single selected note (Maybe I missed some dialogs..)

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 avatar Feb 17 '22 11:02 DmitryArefiev

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

DmitryArefiev avatar Sep 02 '22 14:09 DmitryArefiev

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!

Tantacrul avatar Sep 02 '22 23:09 Tantacrul

Sure! I understand.

DmitryArefiev avatar Sep 04 '22 10:09 DmitryArefiev

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)

RoeiMeiri9 avatar Oct 01 '22 18:10 RoeiMeiri9

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.)

cbjeukendrup avatar Oct 01 '22 19:10 cbjeukendrup

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.

RoeiMeiri9 avatar Oct 01 '22 19:10 RoeiMeiri9

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).

Tantacrul avatar Oct 01 '22 20:10 Tantacrul

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 avatar Dec 22 '22 06:12 papoteur-mga

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

papoteur-mga avatar Dec 30 '22 19:12 papoteur-mga

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

cbjeukendrup avatar Dec 30 '22 19:12 cbjeukendrup

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

papoteur-mga avatar Dec 30 '22 21:12 papoteur-mga

@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...

cbjeukendrup avatar Jan 03 '23 22:01 cbjeukendrup