Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] Add resizeToAvoidBottomInset to ScaffoldPage #444

Closed
henry2man opened this issue Jul 28, 2022 · 1 comment · Fixed by #447
Closed

[Feature] Add resizeToAvoidBottomInset to ScaffoldPage #444

henry2man opened this issue Jul 28, 2022 · 1 comment · Fixed by #447
Labels
enhancement New feature or request

Comments

@henry2man
Copy link
Contributor

henry2man commented Jul 28, 2022

Is your feature request related to a problem? Please describe.
On mobile, when virtual keyboard is shown the UI doesn't adapt to the constrained viewPadding. In FluentUI we don't have an equivalent for Scaffold.resizeToAvoidBottomInset

Describe the solution you'd like
Add bool? resizeToAvoidBottomInset = true to ScaffoldPage in order to have a proper responsive UI on mobile and/or with virtual keyboards.

Describe alternatives you've considered
In order to have a system-wide behavior I've used this workarround:

// On every route that may show a virtual keyboard
SafeArea(
  child: Padding(
      // Add bottom padding
      padding: EdgeInsets.only(
          bottom: MediaQuery.of(context).viewInsets.bottom),
      child: YourWidget()))

Additional context
This is current behavior:

RPReplay_Final1659002105.mov
@henry2man
Copy link
Contributor Author

Current workaround is this:

// On every route that may show a virtual keyboard
SafeArea(
  child: Padding(
      // Add bottom padding
      padding: EdgeInsets.only(
          bottom: MediaQuery.of(context).viewInsets.bottom),
      child: YourWidget()))

If we add resizeToAvoidBottomInset property we may adapt it to apply existing ScaffoldPage paddings, if any.

@bdlukaa bdlukaa added the enhancement New feature or request label Jul 28, 2022
@bdlukaa bdlukaa mentioned this issue Aug 1, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants