Skip to content

Add persistent visual labels to find input #171777

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

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

Conversation

jjaeggli
Copy link
Contributor

About this change

This change implements persistent visual labels in accordance with WCAG 2.1 success criteria 3.3.2, notably addresses the issue that placeholder text does not provide sufficient labels for input fields as described here w3c/wcag#864 and in other sources. This PR serves as an example of a potential change which could address the issue. It may need further review or discussion with UX. Otherwise, if the change sufficiently addresses the issue, it may be merged.

Similar issues have even been addressed in other Microsoft projects, ex.
microsoft/pxt-microbit#2332

Issue:

#167870

How to test:

  1. cmd-f / ctrl-f to open find widget within the editor pane
  2. shift tab to replace expander and activate to show replace field
  3. use screen readers on supported platforms to verify labels are properly announced

Screenshots:

Electron

Screenshot from 2023-01-18 15-32-29

Web

Screen Shot 2023-01-18 at 4 34 16 PM

@jjaeggli
Copy link
Contributor Author

@microsoft-github-policy-service agree

@isidorn isidorn added the accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues label Feb 3, 2023
@rebornix rebornix added this to the March 2023 milestone Feb 7, 2023
@rebornix rebornix modified the milestones: March 2023, April 2023 Mar 22, 2023
@rebornix rebornix modified the milestones: April 2023, On Deck Apr 26, 2023
@Juan52122
Copy link

About this change

This change implements persistent visual labels in accordance with WCAG 2.1 success criteria 3.3.2, notably addresses the issue that placeholder text does not provide sufficient labels for input fields as described here w3c/wcag#864 and in other sources. This PR serves as an example of a potential change which could address the issue. It may need further review or discussion with UX. Otherwise, if the change sufficiently addresses the issue, it may be merged.

Similar issues have even been addressed in other Microsoft projects, ex. microsoft/pxt-microbit#2332

Issue:

#167870

How to test:

  1. cmd-f / ctrl-f to open find widget within the editor pane
  2. shift tab to replace expander and activate to show replace field
  3. use screen readers on supported platforms to verify labels are properly announced

Screenshots:

Electron

Screenshot from 2023-01-18 15-32-29

Web

Screen Shot 2023-01-18 at 4 34 16 PM

Copy link

@Juan52122 Juan52122 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jjaeggli:persistent-labels

@Juan52122
Copy link

jjaeggli:persistent-labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants