'interactive-widget' meta viewport key not behaving correctly when showing a virtual keyboard (on Android 10)
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
People
(Reporter: dave, Assigned: hiro)
References
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
The interactive-widget
property of viewport
meta tag is not working as expected.
Tested
Browsers
I have tested this in:
- firefox beta for android v131.01a
- firefox nightly for android v132.06b
- chrome for android v128.0.6613.146
OS & Device
- Android Version - 10
- Device - OnePlus 5
- Model - ONEPLUS A5000
Web Pages tested
Explantion of viewport behaviour
Viewport Resize Behaviour by Bramus
- interactive-widget=resizes-visual
- interactive-widget=resizes-content
- interactive-widget=overlays-content
All works as expected in Chrome for Android but not so in Android. All test screenshots can be found here, along with the about:supports data from Firefox Beta & Nightly.
Updated•17 days ago
|
Assignee | ||
Comment 1•15 days ago
|
||
I can reproduce the issue on an Android 10 emulator. My best guess is that this code block is something wrong.
Assignee | ||
Comment 2•15 days ago
|
||
As far as I tested on an Android emulator 10,
insetsCompat.getInsets(WindowInsetsCompat.Type.ime()).bottom
and
insetsCompat.getInsets(WindowInsetsCompat.Type.navigationBars()).bottom
are
same, thus subtracting the bavigationBars' bottom will fail to tell the software
keyboard height.
Updated•15 days ago
|
Comment 4•10 days ago
|
||
bugherder |
Assignee | ||
Comment 5•10 days ago
|
||
Comment on attachment 9427640 [details]
Bug 1920755 - Return WindowInsetsCompat.Type.ime()
directly on Android 10. r?#geckoview-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: Users on Android 10 devices will keep suffering from webcompat issues fixed by the default interactive-widget mode (bug 1916002)
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: See comment 0
- List of other uplifts needed: none
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This change is pretty simple, and only affects on Android 10 devices
- String changes made/needed: none
- Is Android affected?: Yes
Assignee | ||
Updated•10 days ago
|
Comment 6•5 days ago
|
||
Comment on attachment 9427640 [details]
Bug 1920755 - Return WindowInsetsCompat.Type.ime()
directly on Android 10. r?#geckoview-reviewers
Approved for 132.0b4.
Updated•5 days ago
|
Description
•