Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MU4 Issue] Wrong OK/Save/Yes buttons order in dialogs on Windows (uses Mac style) #10505

Closed
DmitryArefiev opened this issue Feb 9, 2022 · 19 comments · Fixed by #19429
Closed
Labels
P2 Priority: Medium

Comments

@DmitryArefiev
Copy link
Contributor

DmitryArefiev commented Feb 9, 2022

Describe the bug
MU4 windows builds now using Mac style for OK/Cancel buttons order in dialogs.
It should be changed to Windows style - OK/Save/Yes buttons should be from the left side

MU4:

MU4
MU4 2
MU4 3

MU3:
MU3
MU3 2
MU3 3

Expected behavior
OK/Save buttons should be from the left side (See MU3 (or other Win app) to compare)

Platform information
Windows 10

Additional context
Doesn't occur in MU3

Some dialogs have the right order:
Style, Page settings, Add/Remove system breaks, Transpose

Another dialogs with the wrong order:
New Score wizard, Export, Score properties, Parts, Reset palette

@DmitryArefiev DmitryArefiev added the P3 Priority: Low label Feb 9, 2022
@Jojo-Schmitz
Copy link
Contributor

I believe this doesn't happen in MU3 because it got fixed there, some time after 3.x got split off of master

@DmitryArefiev DmitryArefiev added this to Needs triage in [MU4.0 ALPHA2] via automation Feb 9, 2022
@DmitryArefiev DmitryArefiev added this to To do in [MU4.0 - GENERAL INTERACTIVE DIALOGS] via automation Feb 9, 2022
@vihaanthora
Copy link
Contributor

Hello @DmitryArefiev, I wanted to know if this is the case with every dialog in MU4 or just the ones that you have shown as examples in the issue?
Thank you

@cbjeukendrup
Copy link
Contributor

There is generally no handling for the different order on Windows. So it will be wrong in all dialogs.

@Jojo-Schmitz
Copy link
Contributor

Well, that depends on how the dialog is made. Only those using OK, CANCEL etc. buttons directly rather than a QDialogButtonBox have it potentially wrong (from the back of my head, might not be 100% correct)

@cbjeukendrup
Copy link
Contributor

Ah, yes, dialogs in QWidgets that are using QDialogButtonBox will be correct. But other QWidget dialogs and in all QML dialogs, it will be wrong.

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Feb 9, 2022

See c9bea53, which fixed this in 3.x (and for 3.6.0), via PR #7084 and also listed in issue #6079, errnously(?) marked as being ported.

@DmitryArefiev
Copy link
Contributor Author

DmitryArefiev commented Feb 9, 2022

Hello @DmitryArefiev, I wanted to know if this is the case with every dialog in MU4 or just the ones that you have shown as examples in the issue? Thank you

@vihaanthora @cbjeukendrup I've found there are some dialogs that have the right order:
Style, Page settings, Add/Remove system breaks, Transpose

Another dialogs with the wrong order:
New Score wizard, Export, Score properties, Parts

(Description updated)

@Jojo-Schmitz
Copy link
Contributor

Jojo-Schmitz commented Feb 9, 2022

Score properties is amongst those I fixed for 3.x ;-)

@HemantAntony
Copy link
Contributor

HemantAntony commented May 2, 2022

@DmitryArefiev it seems like some dialogs also have wrong/different orders for yes and no. For example, Revert to Factory Settings and Reset Palette Dialogs. Worth creating a separate issue for this?

@cbjeukendrup
Copy link
Contributor

@HemantAntony It's indeed worth mentioning, but actually it's just a general problem with more or less all dialog buttons, so one general issue like this is enough.

@DmitryArefiev
Copy link
Contributor Author

@DmitryArefiev it seems like some dialogs also have wrong/different orders for yes and no. For example, Revert to Factory Settings and Reset Palette Dialogs. Worth creating a separate issue for this?

@HemantAntony Thanks for catching! I'll just add those details to the description.

btw I see wrong/different orders for yes and no only for Reset Palette Dialog and not for Revert to Factory Settings

Reset palette

Reset to factory

@DmitryArefiev DmitryArefiev changed the title [MU4 Issue] Wrong OK/Save buttons order in dialogs on Windows (uses Mac style) [MU4 Issue] Wrong OK/Save/Yes buttons order in dialogs on Windows (uses Mac style) May 5, 2022
@DmitryArefiev DmitryArefiev added P2 Priority: Medium and removed P3 Priority: Low labels May 5, 2022
@HemantAntony
Copy link
Contributor

@DmitryArefiev does the Yes/No button order also depend on the OS?

@DmitryArefiev
Copy link
Contributor Author

@DmitryArefiev does the Yes/No button order also depend on the OS?

@HemantAntony Yes,

  • on Mac: 'Yes' should be from the right
  • on Win: 'Yes' should be from the left

Also, you can compare with MU3 on Windows (expected result)

@Tantacrul Tantacrul added P3 Priority: Low and removed P2 Priority: Medium labels May 30, 2022
@vpereverzev vpereverzev moved this from Needs triage to In progress in [MU4.0 BETA1] May 31, 2022
@DmitryArefiev
Copy link
Contributor Author

Fixed in #11646

@Tantacrul Tantacrul removed this from In progress in [MU4.0 BETA1] Oct 18, 2022
@Tantacrul Tantacrul added this to To triage in [MU4.0 RELEASE] via automation Oct 18, 2022
@Tantacrul Tantacrul moved this from To triage to In progress in [MU4.0 RELEASE] Oct 18, 2022
@Tantacrul
Copy link
Contributor

Moving to 4.x. Will reopen when 4.0 is out. There are issues with the PR that need extra work to resolve.

@RomanPudashkin RomanPudashkin removed this from In progress in [MU4.0 RELEASE] Oct 21, 2022
@RomanPudashkin RomanPudashkin added this to To do in 4.x LONGLIST via automation Oct 21, 2022
[MU4.0 - GENERAL INTERACTIVE DIALOGS] automation moved this from To do to Done Oct 21, 2022
4.x LONGLIST automation moved this from To do to Requests Oct 21, 2022
@cbjeukendrup cbjeukendrup reopened this Dec 24, 2022
[MU4.0 - GENERAL INTERACTIVE DIALOGS] automation moved this from Done to In progress Dec 24, 2022
4.x LONGLIST automation moved this from Requests to In progress Dec 24, 2022
@cbjeukendrup
Copy link
Contributor

cbjeukendrup commented Dec 24, 2022

Reopening, otherwise people start filing duplicates because they can't find it (which already happened in #15570, which contains some additional info by the way)

@DmitryArefiev DmitryArefiev removed this from In progress in 4.x LONGLIST Dec 26, 2022
@DmitryArefiev DmitryArefiev added this to To do in 4.x SHORTLIST via automation Dec 26, 2022
@DmitryArefiev DmitryArefiev added P2 Priority: Medium and removed P3 Priority: Low labels Mar 1, 2023
@cbjeukendrup cbjeukendrup moved this from To do to In progress in 4.x SHORTLIST Mar 7, 2023
@Jojo-Schmitz
Copy link
Contributor

Came up again in https://musescore.org/en/node/349101

@Jojo-Schmitz
Copy link
Contributor

Check https://musescore.org/en/node/354139#comment-1205896 how bad this wrong order is recieved by the community

@DmitryArefiev
Copy link
Contributor Author

Fixed in #19429

@DmitryArefiev DmitryArefiev linked a pull request Sep 29, 2023 that will close this issue
Eism added a commit that referenced this issue Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Priority: Medium
Projects
Status: Done
8 participants