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

When an unselected tab is dragged, narrator does not read the Name and Index of the Tab. #3507

Closed
YuliKl opened this issue Oct 28, 2020 · 5 comments
Labels
A11yMediumImpact A11ySev2 Non-blocking for core user tasks or blocking for non-core user tasks. A11yWCAG accessibility Narrator, keyboarding, UIA, etc area-TabView help wanted Issue ideal for external contributors no-issue-activity team-Controls Issue for the Controls team

Comments

@YuliKl
Copy link

YuliKl commented Oct 28, 2020

Describe the bug

Steps to reproduce the bug
Pre-requisite:

  • Turn on Narrator.
  • Set Narrator in Item Mode.

Steps to reproduce the behavior:

  1. Launch XAML Controls Gallery application.
  2. Navigate to "All Controls" button and activate it.
  3. Navigate to "TabView" and activate it.
  4. create at least 3-4 tabs.
  5. Navigate to any unselected tab(Do Not activate this tab item).
  6. Drag the tab using "Alt+Shift+Left/Right arrow" key and observe the narrator reading.

Expected behavior

Narrator should read the Name and Index of the dragged tab, also with the current neighboring tab items.

Screenshots

image

Version Info

Xaml Controls Gallery version 1.2.10.0

Additional context

Copied from internal bug

@YuliKl YuliKl added accessibility Narrator, keyboarding, UIA, etc area-TabView A11yMAS labels Oct 28, 2020
@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Oct 28, 2020
@YuliKl
Copy link
Author

YuliKl commented Oct 28, 2020

Suggestion in the original issue:
For that to happen, you have to appropriately raise PropertyChanged events for structural properties (PositionInSet, for instance) as they change. Right now, tab items in XAML do not.

@marcelwgn
Copy link
Collaborator

Hmm, what currently get's announced is certainly not ideal (something along the lines of "Dropped [item name 1] between [item name 2] and [item name 3]"). The issue however with PropertyChanged is issue #3469 which seems to be an issue with narrator. Would just implementing the property changed be fine, even if narrator currently does not properly handle that? If so, I would be more than happy to work on this.

@StephenLPeters StephenLPeters added team-Controls Issue for the Controls team help wanted Issue ideal for external contributors and removed needs-triage Issue needs to be triaged by the area owners labels Oct 28, 2020
@StephenLPeters
Copy link
Contributor

looking at the internal narrator issue, I think #3469 is specific to numberbox's template setup. I don't think tab view would it that. However if we can't get this working, we shouldn't check in code we cant test. I'm a bit confused what is going on here though, tabview is just a list view, why is the selected item drag drop different from other items.

@marcelwgn
Copy link
Collaborator

Hmm that's a good point. I've tried adding the code below in the OnItemsChanged of the TabView (since these drag/drop operations are implemented as remove item, then readd item). Narrator didn't seem to pick that up.

if (const auto peer = winrt::FrameworkElementAutomationPeer::FromElement(*newItem))
{
    auto positionInSet = winrt::AutomationProperties::GetPositionInSet(*newItem);
    winrt::AutomationProperties::SetPositionInSet(*newItem, args.Index());
    peer.RaisePropertyChangedEvent(winrt::AutomationElementIdentifiers::SizeOfSetProperty(),winrt::box_value(positionInSet),winrt::box_value(args.Index()));
}

Could it be that the selected item works correctly as it changes the selection and thus should announce the changed selection?

@chigy chigy added the A11ySev2 Non-blocking for core user tasks or blocking for non-core user tasks. label Apr 6, 2021
@github-actions
Copy link

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A11yMediumImpact A11ySev2 Non-blocking for core user tasks or blocking for non-core user tasks. A11yWCAG accessibility Narrator, keyboarding, UIA, etc area-TabView help wanted Issue ideal for external contributors no-issue-activity team-Controls Issue for the Controls team
Projects
None yet
Development

No branches or pull requests

5 participants