Skip to content

Added Click Reset Warning#9822

Draft
Ag-Derek wants to merge 10 commits intoIDEMSInternational:masterfrom
Ag-Derek:Adding_Reset_Warning
Draft

Added Click Reset Warning#9822
Ag-Derek wants to merge 10 commits intoIDEMSInternational:masterfrom
Ag-Derek:Adding_Reset_Warning

Conversation

@Ag-Derek
Copy link
Collaborator

Fixes #3709
First implemented a warning message when the user clicks reset, this class message in only implemented in dlgTraits for now.
This is not ready for review

@Ag-Derek
Copy link
Collaborator Author

@lilyclements i think you can have a look at the warning message now (Only implemented on dlgTraits) and let me get the feedback

@lilyclements
Copy link
Contributor

This is good! The only main thing is that we would want a checkbox to say to not show this message again.
Is this something that could be done at the "global" level, like, in the reset itself, or would it be something occurring at each dialog? Changing each dialog sounds like a lot of work.

@Ag-Derek
Copy link
Collaborator Author

Sorry i forgot to add it in the description, there is check in the Edit menu called disable warning message it acts as a global level for disabling the warning message. it also doubles as disabling warning message for when a user delete a column

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clicking Reset giving a warning and maybe adding a warning flag?

2 participants