Conversation
Co-authored-by: jcesarmobile <jcesarmobile@gmail.com>
Co-authored-by: Joey Pender <joey.pender@outsystems.com> Co-authored-by: Pedro Bilro <pedro.gustavo.bilro@outsystems.com> Co-authored-by: Mark Anderson <mark.anderson@outsystems.com>
…#8476) Co-authored-by: jcesarmobile <jcesarmobile@gmail.com>
…am#8492) Co-authored-by: Mark Anderson <mark.anderson@outsystems.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
…-team#8271) Co-authored-by: Eric Horodyski <horodyski@ionic.io>
…ermissions (ionic-team#8400) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Pedro Bilro <pedro.gustavo.bilro@outsystems.com>
On API 29 the window is already resized by the system when the keyboard opens (windowSoftInputMode="adjustResize"), and WindowInsetsCompat can only approximate the ime() inset there. SystemBars additionally applied that approximated height as bottom padding, so the WebView was shrunk twice and collapsed to a few hundred pixels whenever WebView >= 140 and viewport-fit=cover were in use. After a rotation with the keyboard open the stale padding stuck until the app was restarted. Only apply the IME padding on API 30+, where ime() reports the real overlap. Verified on a Huawei MAR-LX1A (Android 10, WebView 152): window.innerHeight with the keyboard open is now 442 instead of 141 CSS px, and rotating with the keyboard open no longer leaves the WebView stuck. Behaviour on API 30+ is unchanged. Fixes ionic-team#8601 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…red conflicts)
|
Git applied the upstream-preferred strategy to resolve this sync. Please review the branch carefully before merging. |
|
Warning Review limit reachedNext included review available in 4 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (45)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Beta npm buildMaintainers can publish one Capacitor Plus workspace package from this PR to npm for fast testing. Comment Examples: /publish-beta core
/publish-beta cli
/publish-beta @capacitor-plus/coreIf exactly one workspace package changed, Packages:
The workflow will:
Security note: beta publish is only enabled for branches inside this repository. |
Merge Conflict Review Required
The sync of upstream PR ionic-team#8606 from @ruffzy encountered merge conflicts.
Original PR: ionic-team#8606
What happened
Synced from upstream by Capacitor+ Bot
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.