Create helper function to show custom modal dialogs safer#15265
Merged
Conversation
See test results for failed build of commit dac694c330 |
michaelDCurran
approved these changes
Aug 7, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Link to issue number:
Raised in #15246 (comment)
Summary of the issue:
Sometimes a custom modal dialog is required, instead of just using
gui.message.messageBox.For example, adding additional buttons or controls instead of just message text and ok/cancel/yes/no buttons.
However
gui.message.messageBoxadds special handling to ensure users are warned if actions are blocked by an open modal dialog.Description of user facing changes
Users are warned when trying to perform a task (e.g opening the NVDA menu) when modal dialogs from the add-on store are waiting a response.
Other modal dialogs have also been addressed, unless they are non-blocking (i.e. started in a separate thread).
These might be worth addressing too, on a case-by-case basis.
Description of development approach
Create a helper function to show a dialog as a modal dialog, while performing the necessary handling of
messageBoxTesting strategy:
Test opening various Add-on store dialogs and performing
nvda+nKnown issues with pull request:
There are other dialogs in NVDA with this problem, however most are addressed in this PR.
Change log entries:
N/A
Code Review Checklist: