Skip to content

Identify CheckBox and RadioButton as such#14087

Merged
goderbauer merged 2 commits intoflutter:masterfrom
goderbauer:checkbox
Jan 17, 2018
Merged

Identify CheckBox and RadioButton as such#14087
goderbauer merged 2 commits intoflutter:masterfrom
goderbauer:checkbox

Conversation

@goderbauer
Copy link
Member

Also fixes problems with duplicated nodes for Radio and properly identifies these elements as enabled/disabled.

Fixes #14071.

@goderbauer goderbauer changed the title Identify CHeckBox and RadioButton as such Identify CheckBox and RadioButton as such Jan 12, 2018
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: typo "mutally". Also this sentence just repeats the name of the field. Could you expand the description? For example, you could say that the owning RenderObject corresponds to UI that allows the user to pick one of several mutually exclusive options. The example below would back it up perfectly.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good idea. Done.

@yjbanov
Copy link
Contributor

yjbanov commented Jan 16, 2018

lgtm

@goderbauer goderbauer merged commit 15af864 into flutter:master Jan 17, 2018
@goderbauer goderbauer deleted the checkbox branch January 17, 2018 18:23
@goderbauer goderbauer added the a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) label Mar 5, 2018
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
* Identify CHeckBox and RadioButton as such

* review feedback
engine-flutter-autoroll added a commit that referenced this pull request Dec 4, 2019
[email protected]:flutter/engine.git/compare/1a96087747ac...6179380

git log 1a96087..6179380 --first-parent --oneline
2019-12-03 [email protected] Update fallback to Roboto url to be HTTPS (#14086)
2019-12-03 [email protected] Revert "Fix fml_unittests (#14062)" (#14087)
2019-12-03 [email protected] Revert PRs to unblock David and Jim's work (#14088)
2019-12-03 [email protected] Roll fuchsia/sdk/core/mac-amd64 from 6QxEZ... to OSk8h... (#14085)
2019-12-03 [email protected] Roll src/third_party/dart e4344a568f..a4d799c402 (7 commits) (#14083)


If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected] on the revert to ensure that a human
is aware of the problem.

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Android A11Y: Radio button is not recognized by screen reader

3 participants