-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Improved Confirmation Box for Raw Parameter Setup #3404
Comments
I’ve submitted a pull request to address the issue Enhance full parameter change message box #3412. This PR improves the confirmation dialog by displaying all changed parameters along with their previous and new values, enhancing user verification before finalizing changes. |
In your proposal above you said "This would allow users to verify all changes before confirming." However you only receive this message AFTER you have made the changes. Might it not be a bit superfluous by then ? |
Good point, I have made another commit with added changes in the same PR The following changed are done in the final commitIn a separate confirmation box for parameter changes :
In the saved parameter box :
Preview
|
Issue Details
Enhancement Proposal: Improved Confirmation Box for Raw Parameter Setup
When changing parameters and writing them, the current implementation displays a confirmation dialog box. However, I propose an enhancement to this dialog: it should list all changed parameters, showing both their previous and new values. This would allow users to verify all changes before confirming.
Implementation
I am interested in working on this enhancement myself, if possible. I believe the relevant code may reside in the
GCSViews/ConfigurationView/ConfigRawParams.cs
file, but I’m not entirely certain.Any suggestions or guidance on where to start would be greatly appreciated.
The text was updated successfully, but these errors were encountered: