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

[AdvancedPaste] Fixed issues with Screen Reader integration #36778

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

drawbyperpetual
Copy link
Collaborator

Summary of the Pull Request

Fixes a number of accessibility issues on the Advanced Paste window.

PR Checklist

Detailed Description of the Pull Request / Additional comments

  • Paste actions now have their text and Ctrl+Num shortcut correctly read out by screen readers.
  • Paste actions now work with keyboard navigation (arrow keys, enter, space).
  • The following items now also work correctly with screen readers:
    • Clipboard history items (including images)
    • Clipboard history more options button
    • Thumbs up/down buttons
  • Many other extra description texts have been provided which do not seem to have functional impact when using Narrator in default mode, but will possibly improve descriptions in higher verbosity settings.
  • Paste Formats now use a PasteFormatTemplateSelector that more explicitly indicates disabled items.

Validation Steps Performed

Tested changes using Narrator (default settings) and performed a sanity check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant