Skip to content

Scaffold should respect window.padding.bottom - #1330

Merged
abarth merged 1 commit into
flutter:masterfrom
abarth:bottom_padding
Jan 21, 2016
Merged

abarth merged 1 commit into
flutter:masterfrom
abarth:bottom_padding

Conversation

@abarth

@abarth abarth commented Jan 21, 2016

Copy link
Copy Markdown
Contributor

The space for the keyboard is now represented as bottom padding for the window.
By teaching the scaffold to respect the bottom window padding, we move the
floating action button and snackbars out of the way of the keyboard.

This currently works on Android. I'll need to see how to get the keyboard
geometry on iOS for a similar effect.

Fixes #103

@abarth

abarth commented Jan 21, 2016

Copy link
Copy Markdown
Contributor Author

@chinmaygarde

@chinmaygarde

Copy link
Copy Markdown
Contributor

lgtm. iOS does not communicate the bottom padding to the dart side currently. I will look into using UIKeyboardDidShowNotification and its UIKeyboardFrameBeginUserInfoKey and UIKeyboardFrameEndUserInfoKey keys to achieve the same effect.

@abarth

abarth commented Jan 21, 2016

Copy link
Copy Markdown
Contributor Author

Thanks. I can also look into the iOS side. I need to tech up on that.

The space for the keyboard is now represented as bottom padding for the window.
By teaching the scaffold to respect the bottom window padding, we move the
floating action button and snackbars out of the way of the keyboard.

This currently works on Android. I'll need to see how to get the keyboard
geometry on iOS for a similar effect.

Fixes flutter#103
abarth added a commit that referenced this pull request Jan 21, 2016
Scaffold should respect window.padding.bottom
@abarth
abarth merged commit d10c562 into flutter:master Jan 21, 2016
@abarth
abarth deleted the bottom_padding branch January 21, 2016 21:36
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 17, 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.

Scroll list height doesn't get adjusted when the keyboard is visible

2 participants