Skip to content
This repository was archived by the owner on Jun 13, 2024. It is now read-only.

Fix floating action button tag conflict in Reply (#316)#318

Merged
pennzht merged 2 commits intoflutter:masterfrom
pennzht:fix-reply-study
Sep 21, 2020
Merged

Fix floating action button tag conflict in Reply (#316)#318
pennzht merged 2 commits intoflutter:masterfrom
pennzht:fix-reply-study

Conversation

@pennzht
Copy link
Member

@pennzht pennzht commented Sep 21, 2020

Please review this PR carefully and make sure it does not change any existing behavior/logic/animations.

Thank you!

Description

Add a heroTag for the "Back" floating action button, so that the two floating action buttons ("Back" and _ReplyFab) do not share the same tag (_DefaultHeroTag).

Issue

Closes #316

Copy link
Contributor

@JoseAlba JoseAlba left a comment

Choose a reason for hiding this comment

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

██╗      ██████╗ ████████╗███╗   ███╗
██║     ██╔════╝ ╚══██╔══╝████╗ ████║
██║     ██║  ███╗   ██║   ██╔████╔██║
██║     ██║   ██║   ██║   ██║╚██╔╝██║
███████╗╚██████╔╝   ██║   ██║ ╚═╝ ██║
╚══════╝ ╚═════╝    ╚═╝   ╚═╝     ╚═╝

@pennzht pennzht merged commit 460dd20 into flutter:master Sep 21, 2020
@pennzht pennzht deleted the fix-reply-study branch September 21, 2020 08:26
gspencergoog pushed a commit to gspencergoog/gallery that referenced this pull request Sep 23, 2020
* Move gallery/gallery/ content to gallery/

* Update banner_demo.dart

* Update documentation

* Add hidden files

* Edit Travis script

* Update top level docs

* Update README.md path

* Move Crane middle tab up


Former-commit-id: 13c3bbc
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.

Exception thrown when exiting the Reply study

3 participants