Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Order of OK and Cancel buttons is reversed #4091

Closed
redmunds opened this issue Jun 4, 2013 · 9 comments
Closed

Order of OK and Cancel buttons is reversed #4091

redmunds opened this issue Jun 4, 2013 · 9 comments
Assignees
Milestone

Comments

@redmunds
Copy link
Contributor

redmunds commented Jun 4, 2013

Before the update, dialog buttons were right-aligned, so you had to specify them in the reverse order of how you wanted them to appear. That seems to have changed (although buttons are still right-aligned) because the order of buttons is the reverse of what it was before.

Also, there is no longer any margin between buttons.

@njx
Copy link
Contributor

njx commented Jun 4, 2013

Hmmm, interesting. Right now, it looks like on the Mac the buttons in the Project Settings dialog are in the right order (Cancel / OK), but they're in the wrong order in the Install from URL dialog (Install / Cancel). So there's some additional inconsistency in how we're setting up the order.

@TomMalbran
Copy link
Contributor

The Dialogs Buttons order was changed by my request #3086 in order to fix #3077 and #1089. The buttons are aligned to the right instead of floated, so that they can be added in the correct order, and in Windows the order of the Cancel/OK is reversed by floating them to the right. It seems like I missed a change to the Install from Url Dialog.

@njx
Copy link
Contributor

njx commented Jun 4, 2013

Ah, ok. @redmunds - do the buttons in all the other dialogs (besides Install from URL) look correct on Windows?

@redmunds
Copy link
Contributor Author

redmunds commented Jun 4, 2013

I'm also seeing it in prompts for unsaved changes for single and multiple files, so it seems to effect all templated modal dialogs that have both OK and Cancel buttons.

@ghost ghost assigned jasonsanjose Jun 5, 2013
@jasonsanjose
Copy link
Member

I removed the [framework-update] tag in the title now that we've narrowed it down to the Dialogs API changes. Assigned @jasonsanjose.

@TomMalbran
Copy link
Contributor

Some notes about this issue:

  • The change of the order of the buttons was intentional to fix the issues mentioned above.
  • The buttons in the Install from Url Dialog are now in the wrong order and need to be fixed. I checked with the other dialogs and they all seem to be ok. The fix for this is really simple.
  • Since only the second and third buttons have a margin left, on some dialogs with just 2 right buttons like the Project Settings one, the first button doesn't get a margin left, so when inverted there is no space between the buttons.

From what I get, fixing this last 2 things would fix this issue, so I'll try to make a pull request for that.

@redmunds
Copy link
Contributor Author

redmunds commented Jun 6, 2013

The buttons in the Install from Url Dialog are now in the wrong order and need to be fixed.
I checked with the other dialogs and they all seem to be ok.

Maybe I misunderstood, but all of the dialogs that I mentioned are still have the ok/cancel buttons in the wrong order. I also noticed the buttons in the Remove Extension info message dilaog are also in the wrong order.

@TomMalbran
Copy link
Contributor

Right now on Mac the order of the buttons is normal then primary (Cancel and then OK), that is the same order as buttons in the templates. To fix issue #1089, on Windows the order of the buttons is primary and then normal (OK and then Cancel). So in windows all the buttons should be switched and on Mac should be as before. The issue #1089 only mentioned the save buttons, but the order of the buttons on Windows is usually Accept and then Cancel, so I fixed it for all the buttons.

@jasonsanjose
Copy link
Member

Confirmed fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants