Skip to content

Fixed leading app bar button width constraint - #5802

Merged
HansMuller merged 1 commit into
flutter:masterfrom
HansMuller:app_bar_button
Sep 9, 2016
Merged

HansMuller merged 1 commit into
flutter:masterfrom
HansMuller:app_bar_button

Conversation

@HansMuller

Copy link
Copy Markdown
Contributor

The width constraint for the leading app bar button should be as big as an icon button (not an icon).

@dragostis

Copy link
Copy Markdown
Contributor

LGTM.

@HansMuller

Copy link
Copy Markdown
Contributor Author

Fixes #5804

@HansMuller
HansMuller merged commit 59739ab into flutter:master Sep 9, 2016
@HansMuller
HansMuller deleted the app_bar_button branch September 9, 2016 22:12
@eseidelGoogle

Copy link
Copy Markdown
Contributor

Can/should we test this?

@HansMuller

Copy link
Copy Markdown
Contributor Author

Yes we should test this. #5811

rmacnak-google added a commit that referenced this pull request Jul 19, 2018
482469b Roll Dart to 937ee2e8ca4b76499e24cd463f07bfb736bccd74. (#5800)
bb323c1 Revert "Roll src/third_party/skia 9e0d7e4072e4..297c3c8e85bd (62 commits) (#5802)" (#5803)
0bf69f0 Roll src/third_party/skia 9e0d7e4072e4..297c3c8e85bd (62 commits) (#5802)
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request Jul 19, 2018
482469b Roll Dart to 937ee2e8ca4b76499e24cd463f07bfb736bccd74. (flutter#5800)
bb323c1 Revert "Roll src/third_party/skia 9e0d7e4072e4..297c3c8e85bd (62 commits) (flutter#5802)" (flutter#5803)
0bf69f0 Roll src/third_party/skia 9e0d7e4072e4..297c3c8e85bd (62 commits) (flutter#5802)
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 15, 2021
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.

3 participants