Closed Bug 1091683 Opened 10 years ago Closed 10 years ago

Google inbox page renders with wrong viewport

Categories

(Web Compatibility :: Site Reports, defect)

Product:
Web Compatibility
Component:
Site Reports
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED DUPLICATE of bug 1071620

People

(Reporter: wesj, Unassigned)

References

Details

(Whiteboard: [country-all][notcontactready]) 

The Google Inbox page renders as if there's no viewport on the page at all.

https://www.google.com/inbox/

I think this is likely our bug and not a web-compat one.
Actually, maybe I take that back. The <html> element has a min-width of 720px, but the viewport specifies width=device-width. We layout the html at 720px and the page at 360px like we should. I'll kick this over to webcompat and see if they know why its working in Chrome but not us.
Component: General → Mobile
Product: Firefox for Android → Tech Evangelism
Based on the following rule, I'm guessing there's a bug in their mobile vs. tablet detection: 

html.tablet, html.desktop {
    min-width: 720px;
}

Chrome Mobile gets <html class="... mobile">

Looking for the bug now.
Whiteboard: [country-all][notcontactready]
OK, I think this is probably a dupe of Bug 1071620. The form-factor class is set here:

Wf.prototype.od = function (a) {
var b = 'desktop',
c;
if (c = a) a: {
  for (var d in Ce) if (Ce[d] == a) {
    c = !0;
    break a
  }
  c = !1
}
if (c) return a;
if (Z.userAgent.Zc || Z.userAgent.Ub) if (b = 'mobile', a = !1, 90 == window.orientation || - 90 == window.orientation ? (a = !0, Z.userAgent.Ub && screen.width < screen.height && (a = !1))  : Z.userAgent.Ub && screen.width > screen.height && (a = !0), a = a ? 'landscape' : 'portrait', c = window.innerWidth, 'landscape' === a && 1024 <= c || 'portrait' === a && 768 <= c) b = 'tablet';
return b
};

Everything runs as expected until the end at

> c = window.innerWidth
> 'portrait' === a && 768 <= c

When stepping through c is 980. a was set to 'portrait' and 768 is less than 980.... so b is set to 'tablet'.

If I manually check window.innerWidth it's 360, so presumably we're back to Bug 1071620.

(Note that screen.width is always 360, it doesn't suffer the same bug as window.innerWidth).
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Product: Tech Evangelism → Web Compatibility
Component: Mobile → Site Reports
You need to log in before you can comment on or make changes to this bug.