Skip to content
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

Ensure indeterminate progress dialogs centred on screen before visible #14024

Merged
merged 1 commit into from Aug 30, 2022

Conversation

seanbudd
Copy link
Member

@seanbudd seanbudd commented Aug 17, 2022

Link to issue number:

None

Summary of the issue:

The NVDA update checker and other indeterminate progress dialogs become visible before they are centred on screen.

Description of user facing changes

Indeterminate progress dialogs used in NVDAs GUI are centred before they become visible, and do not jump across the screen.

Description of development approach

Change the order of raise and centre on screen

Testing strategy:

  1. Download and install the build from this PR
  2. Note that the "Updating NVDA dialog" appears centred on screen and does not jump across.

Known issues with pull request:

None

Change log entries:

Not required

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English

@seanbudd seanbudd requested a review from a team as a code owner August 17, 2022 00:33
@AppVeyorBot
Copy link

See test results for failed build of commit 6bfe038668

@seanbudd
Copy link
Member Author

seanbudd commented Aug 17, 2022

@k-kolev1985 can you confirm this PR build fixes the problem in #12192?

@k-kolev1985
Copy link

I updated from an alpha version of NVDA to this PR via the provided installer. The NVDA update installation progress dialog was still not centered on the screen - it was in the top left part of the screen. Did I do something wrong?

@seanbudd
Copy link
Member Author

seanbudd commented Aug 18, 2022

No, those steps are correct.

I thought this would fix the problem as the "Updating NVDA" dialog uses Indeterminate progress dialogs.
It seems that these dialogs cannot be centred before becoming visible as init makes them visible.

I wonder if #13254 fixes this for you

@seanbudd seanbudd marked this pull request as draft August 18, 2022 04:15
@seanbudd seanbudd added the merge-early Merge Early in a developer cycle label Aug 19, 2022
@seanbudd seanbudd marked this pull request as ready for review August 22, 2022 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-early Merge Early in a developer cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants