Treat viewport meta tag as an empty tag if there is no valid content
Categories
(Core :: Layout, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox-esr68 | --- | fixed |
firefox68 | --- | wontfix |
firefox69 | --- | fixed |
firefox70 | --- | fixed |
People
(Reporter: hiro, Assigned: hiro)
References
(Blocks 1 open bug, Regression, )
Details
(Keywords: regression)
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-esr68+
|
Details | Review |
+++ This bug was initially created as a clone of Bug #1556275 +++
The site reported in https://webcompat.com/issues/31296 has <meta name="viewport" content="1200“ />
and the meta viewport is invalid so we should treat it as ViewportType::Empty.
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Depends on D37328
Comment 4•6 years ago
|
||
bugherder |
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Comment 5•6 years ago
|
||
Is this something which should be considered for backport?
Assignee | ||
Comment 6•6 years ago
•
|
||
Comment on attachment 9076658 [details]
Bug 1564021 - Use the default desktop width (980px) in the case where we have any viewport meta element but no valid content there. r?botond
Beta/Release Uplift Approval Request
- User impact if declined: User will see some sites are rendered in an unexpected narrow device width so that font-sizes in the contents might be tremendously bigger than expected or some images gets smaller than expected
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: 1564021 (already uplifted)
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The change is basically trivial, it shouldn't cause any critical issue.
- String changes made/needed: None
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: On mobile this is also an important fix which was a fallout from bug 1500314
- User impact if declined: Same as above
- Fix Landed on Version: 70
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Same as above
- String or UUID changes made by this patch: None
Assignee | ||
Comment 7•6 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #5)
Is this something which should be considered for backport?
Unfortunately yes. I found https://webcompat.com/issues/34891 which is caused by this, there might be other webcompat issues I am not aware of, and of course there are more sites affected by this in the wild.
Comment 8•6 years ago
|
||
Comment on attachment 9076658 [details]
Bug 1564021 - Use the default desktop width (980px) in the case where we have any viewport meta element but no valid content there. r?botond
Fixes some mobile web compat regressions. Approved for GV69 and Fennec 68.1b3.
Comment 9•6 years ago
|
||
bugherder uplift |
![]() |
||
Comment 10•6 years ago
|
||
bugherder uplift |
Updated•3 years ago
|
Description
•