Skip to content

F
Front-End
Unverified Commit cded7efb authored by xster's avatar xster Committed by GitHub
Browse files
Options

Fix CupertinoSwitch accessibility (#19927)

parent dfe5b591
Hide whitespace changes
Inline Side-by-side
Showing with 46 additions and 36 deletions
examples/flutter_gallery/lib/demo/cupertino/cupertino_switch_demo.dart
View file @ cded7efb
...@@ -26,42 +26,51 @@ class _CupertinoSwitchDemoState extends State<CupertinoSwitchDemo> { ...@@ -26,42 +26,51 @@ class _CupertinoSwitchDemoState extends State<CupertinoSwitchDemo> {
child: new Column( child: new Column(
mainAxisAlignment: MainAxisAlignment.spaceAround, mainAxisAlignment: MainAxisAlignment.spaceAround,
children: <Widget>[ children: <Widget>[
new Column( new Semantics(
children: <Widget>[ container: true,
new CupertinoSwitch( child: new Column(
value: _switchValue, children: <Widget>[
onChanged: (bool value) { new CupertinoSwitch(
setState(() { value: _switchValue,
_switchValue = value; onChanged: (bool value) {
}); setState(() {
}, _switchValue = value;
), });
const Text( },
'Active' ),
), const Text(
], 'Active'
),
],
),
), ),
new Column( new Semantics(
children: const <Widget>[ container: true,
const CupertinoSwitch( child: new Column(
value: true, children: const <Widget>[
onChanged: null, const CupertinoSwitch(
), value: true,
const Text( onChanged: null,
'Disabled' ),
), const Text(
], 'Disabled'
),
],
),
), ),
new Column( new Semantics(
children: const <Widget>[ container: true,
const CupertinoSwitch( child: new Column(
value: false, children: const <Widget>[
onChanged: null, const CupertinoSwitch(
), value: false,
const Text( onChanged: null,
'Disabled' ),
), const Text(
], 'Disabled'
),
],
),
), ),
], ],
), ),
......
packages/flutter/lib/src/cupertino/switch.dart
View file @ cded7efb
...@@ -380,10 +380,11 @@ class _RenderCupertinoSwitch extends RenderConstrainedBox { ...@@ -380,10 +380,11 @@ class _RenderCupertinoSwitch extends RenderConstrainedBox {
void describeSemanticsConfiguration(SemanticsConfiguration config) { void describeSemanticsConfiguration(SemanticsConfiguration config) {
super.describeSemanticsConfiguration(config); super.describeSemanticsConfiguration(config);
config.isSemanticBoundary = isInteractive;
if (isInteractive) if (isInteractive)
config.onTap = _handleTap; config.onTap = _handleTap;
config.isChecked = _value;
config.isEnabled = isInteractive;
config.isToggled = _value;
} }
final CupertinoThumbPainter _thumbPainter = new CupertinoThumbPainter(); final CupertinoThumbPainter _thumbPainter = new CupertinoThumbPainter();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment

HIAST - GitLab