ionic-team / ionic-team/ionic-framework

bug: scroll assist does not prevent default browser adjustment in sheet modal

Open
#27,493 11 comments 0 reactions 0 assignees View on GitHub
package: core type: bug
Dominant language
TypeScript
Stars
52.7k
Forks
13.3k
Avg merge
1d 15h
Merged PRs (30d)
51

Description

### Prerequisites

- [X] I have read the [Contributing Guidelines](https://github.com/ionic-team/ionic-framework/blob/main/.github/CONTRIBUTING.md#creating-an-issue).
- [X] I agree to follow the [Code of Conduct](https://ionicframework.com/code-of-conduct).
- [X] I have searched for [existing issues](https://github.com/ionic-team/ionic-framework/issues) that already report this problem, without success.

### Ionic Framework Version

v7.x

### Current Behavior

The inputs when opened inside a sheetModal, get covered by the keyboard
The expected behavior is for the content to move up to make space for the keyboard so the activated input is displayed and not covered

I'm including a video of the first test using the ionic starter-app taken from my cellphone..

https://github.com/ionic-team/ionic-framework/assets/105754419/868430bb-59c3-4f44-b2a1-8ae56eccace9

Then a video with stackblitz running on the chrome inspector showing the same behavior. I couldn't boot stackblitz on the cellphone but //fortunately// the behavior shows up on inspector. (fortunately meaning hopefully easier to debug)

https://github.com/ionic-team/ionic-framework/assets/105754419/2bffb7aa-8969-47f7-bbf9-8e23df0c4723

### Expected Behavior

Scroll Assist should auto scroll / create space to allow for the input to not being hidden behind the keyboard

### Steps to Reproduce

The stackblitz url i'm including followed the described steps but adds some additional code, to show the correct behavior on a regular modal and on a regular page

1. Create a new stackblitz starting with ion-modal controllers https://ionicframework.com/docs/api/modal#controller-modals
2. Add the breakpoints:[0.5] option to the modalController.create() command
3. Add 250px margin-top to the div containing the ion-input element in the modal-example.html
4. Verify bad behavior.

### Code Reproduction URL

https://stackblitz.com/edit/angular-cjmmkn

### Ionic Info

n/a.. running on stackblitz on latest

https://angular-cjmmkn.stackblitz.io/

### Additional Information

This was recently covered on a recent bugfix:
https://github.com/ionic-team/ionic-framework/pull/25849
by @liamdebeasi

But I'm guessing the sheetModal component is breaking the functionality.

There are some more scroll assist issues posted, but nothing really related to this specific case..

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.