Skip to content

Conversation

@jonahwilliams
Copy link
Contributor

@jonahwilliams jonahwilliams commented Sep 12, 2018

Separates the error/helper text and counter from the text field.

When the text field has focus, the counter acts as a live region and updates the remaining characters after typing:

a___ 1/4
"3 characters remaining"
ab__ 2/4
"2 characters remaining"

When an error appears below a text field, this error will be read out once instead of using AccessibilityNodeInfo.setContentInvalid - this will read out "content invalid" after every character press instead of just once.

The error will potentially need to be made an assertive live region, pending specs

@jonahwilliams jonahwilliams changed the title Make helper and error text separate widgets, make error live region Make helper and error text separate widgets, make error and counter live region Sep 12, 2018
@yjbanov
Copy link
Contributor

yjbanov commented Sep 18, 2018

lgtm

@jonahwilliams jonahwilliams merged commit bf7c270 into flutter:master Sep 18, 2018
hereisderek added a commit to hereisderek/flutter that referenced this pull request Sep 18, 2018
…dle-fix

* commit '3c22d7b025642abd865160970d39f8c5ba7d3162':
  clarify how to display an image with a specific density (flutter#21152)
  Roll engine 9e315e6..2e8e96f (2 commits) (flutter#22005)
  Make helper and error text separate widgets, make error and counter live region (flutter#21752)
  Use Xcode legacy build system for iOS builds (flutter#21901) (flutter#21994)
  Run logcat through shell (flutter#21977)
  Add back screenshot extension that was temporarily disabled. (flutter#21828)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants