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.

Sign up for GitHub

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

Jump to bottom

md-input does not label the input with the placeholder text when using VoiceOver #512

Closed
jelbourn opened this issue May 24, 2016 · 2 comments · Fixed by #515
Closed

md-input does not label the input with the placeholder text when using VoiceOver #512

jelbourn opened this issue May 24, 2016 · 2 comments · Fixed by #515
Assignees
@hansl
Labels
Accessibility This issue is related to accessibility (a11y) P1 Impacts a large percentage of users; if a workaround exists it is partial or overly painful
Milestone
alpha.6

Comments

@jelbourn
Copy link
Member

When visiting our demo-app page for md-input using VoiceOver on OSX, the placeholder text for the inputs is not used to label the inputs. When moving focus to the <input>, the reader simple announces "edit text blank". This occurs with Safari, Chrome, and Firefox.
@jelbourn jelbourn added P1 Impacts a large percentage of users; if a workaround exists it is partial or overly painful Accessibility This issue is related to accessibility (a11y) labels May 24, 2016
@jelbourn jelbourn added this to the alpha.6 milestone May 24, 2016
@hansl
Copy link
Contributor

hansl commented May 25, 2016

be5e93a didn't change the for attribute of the label :(

hansl added a commit to hansl/material2 that referenced this issue May 25, 2016
@hansl
fix: use the right ID for the label.
dfe7a84 
Fixes angular#512.
@hansl hansl mentioned this issue May 25, 2016
Merged
@robertmesserle robertmesserle mentioned this issue May 25, 2016
Closed
jelbourn pushed a commit that referenced this issue May 25, 2016
@hansl @jelbourn
fix(input): use the right ID for the label. (#515)
06aa3f1 
Fixes #512.
andrewseguin pushed a commit to andrewseguin/components that referenced this issue Oct 15, 2018
@benelliott @jelbourn
feat: add summaries to all menu items and category pages (angular#512)
d15b77f
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@hansl hansl

Labels
Accessibility This issue is related to accessibility (a11y) P1 Impacts a large percentage of users; if a workaround exists it is partial or overly painful
Projects
None yet
Milestone
alpha.6
3 participants
@robertmesserle @hansl @jelbourn