Skip to content

Conversation

@HansMuller
Copy link
Contributor

@HansMuller HansMuller commented Mar 5, 2024

Improved the smiley image TextButton example a little. Handling the case where the Future.delayed object that represents the button's one second long action is superseded by a second button press that triggers a new one-second action. This does complicate the example - just a little - but it's a little more robust. In case someone copy-and-pastes the code.

The TextButton example was recently updated: #144583

@github-actions github-actions bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos labels Mar 5, 2024
@HansMuller HansMuller requested a review from QuncCccccc March 5, 2024 19:54
Copy link
Contributor

@QuncCccccc QuncCccccc left a comment

Choose a reason for hiding this comment

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

Thanks for the improvements! LGTM!

nit: Just noticed if this is running on mobiles, the Switches take a lot of space. Maybe another improvement is to replace Card with NavigationDrawer which can be triggered by a leading button on AppBar.😆

@HansMuller
Copy link
Contributor Author

nit: Just noticed if this is running on mobiles, the Switches take a lot of space. Maybe another improvement is to replace Card with NavigationDrawer which can be triggered by a leading button on AppBar.

OK, now I have a reason to make yet another update to this example!

@HansMuller HansMuller added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 5, 2024
@auto-submit auto-submit bot merged commit 52961f7 into flutter:master Mar 5, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Mar 6, 2024
flutter/flutter@3b5a2ec...8f84f3f

2024-03-06 [email protected] Roll Packages from 2aa6e3f to 9b88dbc (8 revisions) (flutter/flutter#144693)
2024-03-06 [email protected] Roll Flutter Engine from 370e7d5866d9 to b6efe0dd88fe (1 revision) (flutter/flutter#144668)
2024-03-06 [email protected] Roll Flutter Engine from d374c78bcf52 to 370e7d5866d9 (1 revision) (flutter/flutter#144661)
2024-03-06 [email protected] Roll Flutter Engine from effcf97a1f7c to d374c78bcf52 (5 revisions) (flutter/flutter#144659)
2024-03-06 [email protected] Fill in SliverConstraints fields missing from ==, hashCode, toString (flutter/flutter#143661)
2024-03-06 [email protected] Roll Flutter Engine from 49bc1577f317 to effcf97a1f7c (10 revisions) (flutter/flutter#144653)
2024-03-05 49699333+dependabot[bot]@users.noreply.github.com Bump codecov/codecov-action from 4.0.2 to 4.1.0 (flutter/flutter#144174)
2024-03-05 [email protected] Fix embedding FlutterMacOS.framework for macOS add2app via cocoapods (flutter/flutter#144248)
2024-03-05 [email protected] Enable asset transformation feature in hot reload workflow (excluding Web) (flutter/flutter#144161)
2024-03-05 [email protected] Roll Flutter Engine from 3e8b0deffe4e to 49bc1577f317 (5 revisions) (flutter/flutter#144639)
2024-03-05 [email protected] Updated the smiley TextButton example again (flutter/flutter#144630)
2024-03-05 [email protected] Adds missing `style` to `PopupMenuButton` (flutter/flutter#143392)
2024-03-05 [email protected] Roll Flutter Engine from a7c785884903 to 3e8b0deffe4e (1 revision) (flutter/flutter#144629)
2024-03-05 [email protected] Add regression test for TabBar crash (flutter/flutter#144627)
2024-03-05 [email protected] Roll Packages from 0625827 to 2aa6e3f (5 revisions) (flutter/flutter#144628)
2024-03-05 [email protected] remove unused `firstBuildTime` parameter in `DevFS::update` (flutter/flutter#144576)
2024-03-05 [email protected] Roll Flutter Engine from 8916bb32b7b8 to a7c785884903 (1 revision) (flutter/flutter#144624)
2024-03-05 [email protected] Revert "_DefaultTabControllerState should dispose all created TabContoller instances. (#136608)" (flutter/flutter#144579)

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

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
LouiseHsu pushed a commit to LouiseHsu/packages that referenced this pull request Mar 7, 2024
flutter/flutter@3b5a2ec...8f84f3f

2024-03-06 [email protected] Roll Packages from 2aa6e3f to 9b88dbc (8 revisions) (flutter/flutter#144693)
2024-03-06 [email protected] Roll Flutter Engine from 370e7d5866d9 to b6efe0dd88fe (1 revision) (flutter/flutter#144668)
2024-03-06 [email protected] Roll Flutter Engine from d374c78bcf52 to 370e7d5866d9 (1 revision) (flutter/flutter#144661)
2024-03-06 [email protected] Roll Flutter Engine from effcf97a1f7c to d374c78bcf52 (5 revisions) (flutter/flutter#144659)
2024-03-06 [email protected] Fill in SliverConstraints fields missing from ==, hashCode, toString (flutter/flutter#143661)
2024-03-06 [email protected] Roll Flutter Engine from 49bc1577f317 to effcf97a1f7c (10 revisions) (flutter/flutter#144653)
2024-03-05 49699333+dependabot[bot]@users.noreply.github.com Bump codecov/codecov-action from 4.0.2 to 4.1.0 (flutter/flutter#144174)
2024-03-05 [email protected] Fix embedding FlutterMacOS.framework for macOS add2app via cocoapods (flutter/flutter#144248)
2024-03-05 [email protected] Enable asset transformation feature in hot reload workflow (excluding Web) (flutter/flutter#144161)
2024-03-05 [email protected] Roll Flutter Engine from 3e8b0deffe4e to 49bc1577f317 (5 revisions) (flutter/flutter#144639)
2024-03-05 [email protected] Updated the smiley TextButton example again (flutter/flutter#144630)
2024-03-05 [email protected] Adds missing `style` to `PopupMenuButton` (flutter/flutter#143392)
2024-03-05 [email protected] Roll Flutter Engine from a7c785884903 to 3e8b0deffe4e (1 revision) (flutter/flutter#144629)
2024-03-05 [email protected] Add regression test for TabBar crash (flutter/flutter#144627)
2024-03-05 [email protected] Roll Packages from 0625827 to 2aa6e3f (5 revisions) (flutter/flutter#144628)
2024-03-05 [email protected] remove unused `firstBuildTime` parameter in `DevFS::update` (flutter/flutter#144576)
2024-03-05 [email protected] Roll Flutter Engine from 8916bb32b7b8 to a7c785884903 (1 revision) (flutter/flutter#144624)
2024-03-05 [email protected] Revert "_DefaultTabControllerState should dispose all created TabContoller instances. (#136608)" (flutter/flutter#144579)

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

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: text input Entering text in a text field or keyboard related problems autosubmit Merge PR when tree becomes green via auto submit App d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants