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

Add-on Store: First press of Ctrl+Tab shouldn't be improperly announced #14986

Closed
XLTechie opened this issue Jun 8, 2023 · 11 comments
Closed
Labels
close/worksforme feature/addon-store Features / behavior of the add-on Store p3 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.

Comments

@XLTechie
Copy link
Collaborator

XLTechie commented Jun 8, 2023

Steps to reproduce:

  • Open the Add-on store.
  • Press Ctrl+Tab.

Actual behavior:

NVDA speaks:

tab control
panel

Expected behavior:

NVDA should speak:

Updatable add-ons  tab  selected

Since that's the tab that has been changed to.

Additional note:

Subsequent presses of Ctrl+Tab do speak the selected tab, but the first one is never spoken.

System configuration

NVDA portable, version alpha-28372,94715243
Microsoft® Windows® Operating System version 10.0.19041.2311

@seanbudd seanbudd added p3 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation. feature/addon-store Features / behavior of the add-on Store labels Jun 8, 2023
@seanbudd
Copy link
Member

seanbudd commented Jun 8, 2023

This could potentially be fixed by always manually focusing the selected tab's control when performing ctrl+tab, but this would make #14987 invalid

@XLTechie
Copy link
Collaborator Author

XLTechie commented Jun 8, 2023 via email

@CyrilleB79
Copy link
Collaborator

CyrilleB79 commented Jun 21, 2023

For reference in Windows Explorer files properties dialog, we have:

  • if an item in the tab page is focused (e.g. Hidden checkbox, or Read-only checkbox), control+tabbing put the focus in the new tab page
  • if the tab header (e.g. General, Shared) is focused, control+tabbing put the focus on the next tab header and not in the tab page.

IMO, this behaviour is a good UX and should be copied in the add-on store.

@CyrilleB79
Copy link
Collaborator

@seanbudd would this PR deserve being integrated in NVDA 2023.2? For now it is not included in its milestone.

Of course it's not a critical bug at all.

However, I imagine that ctrl+tab may be the preferred way to change tab in the store. Leaving this bug in NVDA 2023.2 make the discovery of the store less easy for general users and would also lead to a bad first impression.

@XLTechie
Copy link
Collaborator Author

I agree with what @CyrilleB79 describes in #14986 (comment). This is the desirable behavior.
(Although I believe ctrl+tab is meant in the comment, where alt+tab is used)

@CyrilleB79
Copy link
Collaborator

I agree with what @CyrilleB79 describes in #14986 (comment). This is the desirable behavior. (Although I believe ctrl+tab is meant in the comment, where alt+tab is used)

Thanks. I have updated my previous comment for clarity.

@seanbudd
Copy link
Member

This is a minor bug with no clear solution or easy fix - I don't think it should hold up the release

@ABuffEr
Copy link
Contributor

ABuffEr commented Dec 30, 2023

Hi,
it's not resolved yet. And it'll continue to give a bad impression in incoming 2024.1.
Personally I agree with Cyrille's observations, but add also that Windows Explorer not keeps focus moving among tabs. So I suppose that we'd like to get focus on add-on list, as when opening the store.
This result, though, seems not so trivial to get working, because adding self.addonListView.SetFocus() to page changed handler NVDA reads the list names inconsistently, and nothing else of the updated panel.
By contrast, adding self.addonListTabs.SetFocus() seems to work well, always focusing on tab title (instead of say "tab control panel").
Moreover, due to filters presence before the add-on list, and the fact those always resets their status, I'm not totally convinced that focusing on add-on list when opening the store is the best choice, but it's irrelevant at this stage.

@Adriani90
Copy link
Collaborator

The filter reset issue is not ideal at all, I filled #15947 for that.

In my view it would be an improvement to redirect the focus automatically to the tab area when pressing ctrl+tab if this is the only way of solving this particular issue.

@XLTechie
Copy link
Collaborator Author

XLTechie commented Dec 31, 2023 via email

seanbudd pushed a commit that referenced this issue Jan 4, 2024
See discussion in #14986

Summary of the issue:
Switching tabs in add-on store with ctrl+tab announces "tab control panel", and moves the focus in a strange position.

Description of user facing changes
Users will be positioned on tab list, over the new current tab, that will be read by NVDA.

Description of development approach
Simply SetFocus on addonListTabs (wx.Notebook) if it's not already here, at the end of page change event handler.
Nael-Sayegh pushed a commit to Nael-Sayegh/nvda that referenced this issue Feb 15, 2024
…cess#15988)

See discussion in nvaccess#14986

Summary of the issue:
Switching tabs in add-on store with ctrl+tab announces "tab control panel", and moves the focus in a strange position.

Description of user facing changes
Users will be positioned on tab list, over the new current tab, that will be read by NVDA.

Description of development approach
Simply SetFocus on addonListTabs (wx.Notebook) if it's not already here, at the end of page change event handler.
@Adriani90
Copy link
Collaborator

Closing as fixed in #15988.

SaschaCowley pushed a commit to SaschaCowley/nvda that referenced this issue Feb 27, 2024
…cess#15988)

See discussion in nvaccess#14986

Summary of the issue:
Switching tabs in add-on store with ctrl+tab announces "tab control panel", and moves the focus in a strange position.

Description of user facing changes
Users will be positioned on tab list, over the new current tab, that will be read by NVDA.

Description of development approach
Simply SetFocus on addonListTabs (wx.Notebook) if it's not already here, at the end of page change event handler.
Adriani90 pushed a commit to Adriani90/nvda that referenced this issue Mar 13, 2024
…cess#15988)

See discussion in nvaccess#14986

Summary of the issue:
Switching tabs in add-on store with ctrl+tab announces "tab control panel", and moves the focus in a strange position.

Description of user facing changes
Users will be positioned on tab list, over the new current tab, that will be read by NVDA.

Description of development approach
Simply SetFocus on addonListTabs (wx.Notebook) if it's not already here, at the end of page change event handler.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
close/worksforme feature/addon-store Features / behavior of the add-on Store p3 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.
Projects
None yet
Development

No branches or pull requests

5 participants