Command buttons in settings view are in the wrong order #17565

Closed
opened 2026-01-31 05:46:09 +00:00 by claunia · 3 comments
Owner

Originally created by @danielchalmers on GitHub (May 25, 2022).

Windows Terminal version

1.12.10982.0

Windows build number

10.0.19044.0

Other Software

No response

Steps to reproduce

This is definitely a nitpick but shouldn't the Discard changes and Save buttons in the settings view be the other way round to match the rest of the Windows design?

image

compared to

image

image

image

image

image

Even the Close all tabs dialog in Terminal has the commit button left-to-right 😜

image

More examples: https://docs.microsoft.com/en-us/windows/win32/uxguide/ctrl-command-buttons

Expected Behavior

image

Actual Behavior

image

Originally created by @danielchalmers on GitHub (May 25, 2022). ### Windows Terminal version 1.12.10982.0 ### Windows build number 10.0.19044.0 ### Other Software _No response_ ### Steps to reproduce This is definitely a nitpick but shouldn't the `Discard changes` and `Save` buttons in the settings view be the other way round to match the rest of the Windows design? ![image](https://user-images.githubusercontent.com/7112040/170243991-d0dd806e-db17-41c9-a50d-e194f579c506.png) compared to ![image](https://user-images.githubusercontent.com/7112040/170242004-1daa3553-6387-45ad-98a7-185ae3777519.png) ![image](https://user-images.githubusercontent.com/7112040/170242480-18ac3f12-b156-4c4e-b0e0-71bca356f5d7.png) ![image](https://user-images.githubusercontent.com/7112040/170242525-fe1c9335-4500-4c69-9008-78fcef92e05f.png) ![image](https://user-images.githubusercontent.com/7112040/170242576-5c4dd279-80b0-449e-b86f-c178a79dc39d.png) ![image](https://user-images.githubusercontent.com/7112040/170243643-2ea16c7f-8660-4246-9dbb-42f8c8c81ddb.png) Even the Close all tabs dialog in Terminal has the commit button left-to-right 😜 ![image](https://user-images.githubusercontent.com/7112040/170243240-53ba5e23-e15e-4288-a6a2-f4f99af4b84f.png) More examples: https://docs.microsoft.com/en-us/windows/win32/uxguide/ctrl-command-buttons ### Expected Behavior ![image](https://user-images.githubusercontent.com/7112040/170240822-deaa2f96-68b7-4913-9c84-a620a3903349.png) ### Actual Behavior ![image](https://user-images.githubusercontent.com/7112040/170238936-f6507a54-8215-404d-b609-897872f59aed.png)
Author
Owner

@zadjii-msft commented on GitHub (May 25, 2022):

@cinnamon-msft for UI design feedback

Obviously this is a ship-blocking issue, I don't know how this ever passed review 😝

@zadjii-msft commented on GitHub (May 25, 2022): @cinnamon-msft for UI design feedback Obviously this is a ship-blocking issue, I don't know how this ever passed review 😝
Author
Owner

@zadjii-msft commented on GitHub (Jun 6, 2022):

consensus: we couldn't come up with the reason we went with this one. We could not find a counter example that validated the design we currently have.

We may also just get rid of these for #10627

@zadjii-msft commented on GitHub (Jun 6, 2022): consensus: we couldn't come up with the reason we went with this one. We could not find a counter example that validated the design we currently have. We may also just get rid of these for #10627
Author
Owner

@ghost commented on GitHub (Jul 6, 2022):

:tada:This issue was addressed in #13237, which has now been successfully released as Windows Terminal Preview v1.15.186.🎉

Handy links:

@ghost commented on GitHub (Jul 6, 2022): :tada:This issue was addressed in #13237, which has now been successfully released as `Windows Terminal Preview v1.15.186`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.15.186) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#17565