Skip to content
This repository has been archived by the owner on Feb 16, 2024. It is now read-only.

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

Merged
merged 2 commits into from Sep 21, 2020

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