Skip to content

Settings changed popup - Replace discard with cancel#599

Merged
nullsystem merged 1 commit intoNeotokyoRebuild:masterfrom
nullsystem:GH-590_CancelNotDiscard
Sep 19, 2024
Merged

Settings changed popup - Replace discard with cancel#599
nullsystem merged 1 commit intoNeotokyoRebuild:masterfrom
nullsystem:GH-590_CancelNotDiscard

Conversation

@nullsystem
Copy link
Copy Markdown
Collaborator

Description

  • Basically go back to settings instead, in which settings have a "Restore" button anyway

Toolchain

  • Linux GCC Distro Native Arch/GCC 14

Linked Issues

@nullsystem nullsystem requested a review from a team September 18, 2024 17:27
@nullsystem nullsystem added the Release priority Pull request is a priority for the next release label Sep 18, 2024
@nullsystem nullsystem added this to the v8.1-prealpha milestone Sep 18, 2024
Rainyan
Rainyan previously approved these changes Sep 18, 2024
Copy link
Copy Markdown
Member

@Rainyan Rainyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The "Restore" button in the settings menu should perhaps be renamed "Discard current changes" or similar - I assumed it would restore the defaults, but that does not appear to be the case. This could leave a user confused, where they cannot exit the menu because of pending changes, but they don't want to press the "Restore" button, fearing it'll override everything back to default instead of just discarding the pending changes.

It'd also be nice to have a "Restore defaults" button, but that's not strictly related.

@nullsystem
Copy link
Copy Markdown
Collaborator Author

The "Restore" button in the settings menu should perhaps be renamed "Discard current changes" or similar - I assumed it would restore the defaults, but that does not appear to be the case. This could leave a user confused, where they cannot exit the menu because of pending changes, but they don't want to press the "Restore" button, fearing it'll override everything back to default instead of just discarding the pending changes.

It'd also be nice to have a "Restore defaults" button, but that's not strictly related.

@Rainyan I would like to try to keep the button short though. Other alternatives like "Reset", "Discard", or "Cancel", although both "Discard"/"Cancel" does sound more like it'll also exit the settings page.

@xedmain
Copy link
Copy Markdown
Contributor

xedmain commented Sep 18, 2024

The "Restore" button in the settings menu should perhaps be renamed "Discard current changes" or similar - I assumed it would restore the defaults, but that does not appear to be the case. This could leave a user confused, where they cannot exit the menu because of pending changes, but they don't want to press the "Restore" button, fearing it'll override everything back to default instead of just discarding the pending changes.
It'd also be nice to have a "Restore defaults" button, but that's not strictly related.

@Rainyan I would like to try to keep the button short though. Other alternatives like "Reset", "Discard", or "Cancel", although both "Discard"/"Cancel" does sound more like it'll also exit the settings page.

revert

@xedmain
Copy link
Copy Markdown
Contributor

xedmain commented Sep 18, 2024

about confusing the user, that's why the discard button could be kept, because they may not notice the revert/restore button and will be stuck in the cancel loop or think they can only apply the settings or sth

* Make it 3 buttons, add in "Cancel"
* Rename "Restore" to "Revert"
* NeoUI updated to add spacing for horizontal layout
* fixes NeotokyoRebuild#590
@nullsystem
Copy link
Copy Markdown
Collaborator Author

@xedmain Alright discard back in now

@nullsystem nullsystem requested review from a team and Rainyan September 18, 2024 20:38
Copy link
Copy Markdown
Member

@Rainyan Rainyan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I'm neutral on the "Discard" button for the popup, but the "Revert" wording is much clearer as to what the button does.

@Rainyan Rainyan requested a review from a team September 18, 2024 21:25
@nullsystem nullsystem merged commit 7b222e0 into NeotokyoRebuild:master Sep 19, 2024
@nullsystem nullsystem deleted the GH-590_CancelNotDiscard branch May 7, 2025 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release priority Pull request is a priority for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Cancel button in the Root Menu's Options

4 participants