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

paper-input's error-message is always announced by screen readers #553

Open
3 of 8 tasks
tgsergeant opened this issue Jul 3, 2017 · 1 comment
Open
3 of 8 tasks

Comments

@tgsergeant
Copy link

Screenreaders always announce a paper-input's error message, even when there's no error present.

Expected outcome

The error message shouldn't be announced until there is an error.

Actual outcome

The error message is always announced

Live Demo

https://jsfiddle.net/knL6czt8/

Steps to reproduce

  1. Turn on a screen reader
  2. Select the input field with the screenreader
  3. Listen to the announcement. Will be something like "URL, edit text, invalid URL"

screenshot of chromevox announcing the error text

It's possible that this is expected behavior, since it's possible to phrase the error message in a way that is useful to be announced beforehand, by saying something like "Must be a valid URL" instead of just "Invalid URL". If that's the case, then this should be documented somewhere, since it's quite unexpected and easy to get wrong (see: about a dozen places in Chrome)

Browsers Affected

  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge
  • IE 11
  • IE 10

I've tested with ChromeOS + ChromeVox, Edge + NVDA and Safari + VoiceOver.

Thanks for the help!

@Waiin-T
Copy link

Waiin-T commented Aug 1, 2017

I also have same issue.
While using multiple paper-input in dom-repeat error-message is always visible after some iterations.
And below error is also occurred -

legacy-element-mixin.html:338 Uncaught RangeError: Maximum call stack size exceeded.
at HTMLElement.fire (legacy-element-mixin.html:338)
at HTMLElement.attached (paper-input-addon-behavior.html:25)
at HTMLElement.attached (class.html:214)
at HTMLElement.attached (class.html:212)
at HTMLElement.connectedCallback (legacy-element-mixin.html:83)
at HTMLElement.__ensureInstance (dom-if.html:207)
at HTMLElement.__render (dom-if.html:146)
at Debouncer.__renderDebouncer.Polymer.Debouncer.debounce [as _callback] (dom-if.html:113)
at Debouncer.flush (debounce.html:58)
at flushDebouncers (flush.html:31)…

Browsers Affected

  • Chrome
  • Firefox
  • IE 11

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

No branches or pull requests

3 participants