<!--
Thanks for filing a pull request!
Reviewers are typically assigned within a week of filing a request.
To learn more about code review, see our documentation on Tree Hygiene: https://github....
This is their code, are we overriding the disable style somehow?
```dart
final style =
buttonStyle ??
TextButton.styleFrom(
foregroundColor: theme.colors.primary,
...
I also don't think we should go with this. Sometimes which build context is important.
```dart
Widget _myBuilder(BuildContext context) {
if (TickerMode.of(context)) {
return ImplicitAnimat...
> MediaQueryData.disableAnimations is related to the system accessibility setting
Yes, but what if a user toggle this setting and see all the animation is still animating. That feels bad. Right ...
<!--
Thanks for filing a pull request!
Reviewers are typically assigned within a week of filing a request.
To learn more about code review, see our documentation on Tree Hygiene: https://github....
What is the relationship between iOS Engine, engine shell, and view controller? I thought iOS engine will create both view controller and engine shell so this assertion should not be null I think?
For individual a11y issues, we are working hard on fixing them. it is an ongoing effort, Things like https://github.com/flutter/flutter/issues/173097, https://github.com/flutter/flutter/issues/9960...
looking at the code this `_icon` is just to check whether a padding is needed. The developer are still expect to build an icon in the child by themselves.
might as well let this be `bool addPadddi...
We can consider make this a regular constructor so you don't need to pass "addPaddding" information through the default constructor.
```dart
const OutlinedButton({
...
}) : _addPadding = false;
...