visualViewport values don't take browser zoom into account |
||||
Issue descriptionChrome Version : 51.0.2704.7 OS Version: All What steps will reproduce the problem? 1. Open any page 2. In devtools console, get the value of document.visualViewport.clientWidth 3. Zoom in or out using browser zoom (ctrl+/-) 4. Repeat step 2 What is the expected result? The value should change to account for the zoom level. i.e. If we're at 200% browser zoom, the client width should be half of what it was before we zoomed. This is in line with what window.innerWidth reports. What happens instead of that? The value is unchanged after zooming. UserAgentString: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.7 Safari/537.36
,
Apr 24 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b commit 67d8ccb45177b214dbdb51375f1ed1ba31ffa07b Author: ymalik <ymalik@chromium.org> Date: Sun Apr 24 03:42:53 2016 Update visual viewport API to take browser zoom into account. BUG= 604932 Review URL: https://codereview.chromium.org/1905693003 Cr-Commit-Position: refs/heads/master@{#389398} [add] https://crrev.com/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b/third_party/WebKit/LayoutTests/fast/dom/viewport/viewport-dimensions-under-browser-zoom.html [modify] https://crrev.com/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b/third_party/WebKit/Source/core/frame/VisualViewport.cpp
,
Apr 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b commit 67d8ccb45177b214dbdb51375f1ed1ba31ffa07b Author: ymalik <ymalik@chromium.org> Date: Sun Apr 24 03:42:53 2016 Update visual viewport API to take browser zoom into account. BUG= 604932 Review URL: https://codereview.chromium.org/1905693003 Cr-Commit-Position: refs/heads/master@{#389398} [add] https://crrev.com/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b/third_party/WebKit/LayoutTests/fast/dom/viewport/viewport-dimensions-under-browser-zoom.html [modify] https://crrev.com/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b/third_party/WebKit/Source/core/frame/VisualViewport.cpp
,
Apr 26 2016
,
Jun 23 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by ymalik@chromium.org
, Apr 20 2016Owner: ymalik@chromium.org
Status: Started (was: Assigned)