Skip to content

Make helper and error text separate widgets, make error and counter live region - #21752

Merged
jonahwilliams merged 3 commits into
flutter:masterfrom
jonahwilliams:helper_test
Sep 18, 2018
Merged

jonahwilliams merged 3 commits into
flutter:masterfrom
jonahwilliams:helper_test

Conversation

@jonahwilliams

@jonahwilliams jonahwilliams commented Sep 12, 2018 •

Copy link
Copy Markdown
Contributor

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

yjbanov commented Sep 18, 2018

Copy link
Copy Markdown
Contributor

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