-
-
Notifications
You must be signed in to change notification settings - Fork 614
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
Report position and set size for gridcells in treegrids #16504
Conversation
See test results for failed build of commit 4b1bc354c2 |
@@ -121,6 +121,10 @@ def isDescendantOf(self, obj: "NVDAObjects.NVDAObject") -> bool: | |||
return bool(res) | |||
|
|||
def _get_positionInfo(self): | |||
if self.role == controlTypes.Role.TABLECELL: | |||
for anc in reversed(api.getFocusAncestors()): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is definitely going to return wrong information when the table cell doesn't have focus, such as when using object navigation
I think you want to traverse the parents of the objects instead.
@LeonarddeR Thanks for a good catch; I could have really had egg on my face there. |
Closing as more input is needed on the UX expected. |
Link to issue number:
Fixes #15818
Summary of the issue:
When navigating an ARIA treegrid's cells (as opposed to its rows), the position in set and set size are not supported.
Description of user facing changes
Position information are now reported.
Description of development approach
Modified
Ia2Web._get_positionInfo
to return the closest ancestor treeitem's position information if the current focus is a gridcell.Testing strategy:
Manually tested with the APG Treegrid Email Inbox Example.
Known issues with pull request:
Level information is reported at the end of speech output when navigating by cell, unlike when navigating by row, where it is reported first if the level has changed.
Code Review Checklist: