Skip to content

Ensure that DropDownMenus are always onscreen - #3742

Merged
abarth merged 1 commit into
flutter:masterfrom
abarth:onscreen_drop_down
May 5, 2016
Merged

abarth merged 1 commit into
flutter:masterfrom
abarth:onscreen_drop_down

Conversation

@abarth

@abarth abarth commented May 4, 2016

Copy link
Copy Markdown
Contributor

This patch sizes the menu such that it is always on screen, but doesn't scroll
the menu to ensure that the currently selected item is always visible and on
top of the button. That will need to wait for a later patch.

Also, teach CustomPaint how to repaint animations more efficiently.

Fixes #3720

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Creates a CustomPainter. ...

@Hixie

Hixie commented May 4, 2016

Copy link
Copy Markdown
Contributor

LGTM

@abarth
abarth force-pushed the onscreen_drop_down branch from 17a9317 to c2c4c3b Compare May 4, 2016 22:48
This patch sizes the menu such that it is always on screen, but doesn't scroll
the menu to ensure that the currently selected item is always visible and on
top of the button. That will need to wait for a later patch.

Also, teach CustomPaint how to repaint animations more efficiently.

Fixes flutter#3720
@abarth
abarth force-pushed the onscreen_drop_down branch from c2c4c3b to 4211778 Compare May 4, 2016 23:01
@abarth
abarth merged commit 6a46bf2 into flutter:master May 5, 2016
@abarth
abarth deleted the onscreen_drop_down branch May 5, 2016 00:13
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 16, 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.

Dropdown extends off the top of the screen

2 participants