IntersectionObserver.rootBounds are wrong with navigator zoom != 100%
Reported by
sylvain....@gmail.com,
Sep 21
|
|||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36 Steps to reproduce the problem: 1. Open the testcase with zoom navigator = 100% 2. scroll the main panel up an down 3. the green "Comment..." is always visible : it is detached with positon="fixed" when it became hidden, thanks to the IntersectionObserver api. 4. Change the navigator zoom and scroll again... What is the expected behavior? The "Comment..." panel should be fixed on the top or the bottom of the main area when this main area is scrolled (as with zoom = 100%) What went wrong? The fixed positions computed when the "Comment..." panel is detached are wrong. Did this work before? N/A Does this work in other browsers? N/A Chrome version: 69.0.3497.100 Channel: stable OS Version: 10.0 Flash Version: IntersectionObserver.rootBounds values seems wrong when zoom != 100%. Failed also with Chrome Canary. Version 71.0.3558.0 (Build officiel) canary (32 bits)
,
Sep 24
Able to reproduce the issue on Mac 10.13.3, Win-10 and Ubuntu 14.04 using chrome reported version #69.0.3497.100 and latest canary #71.0.3558.0. This is a non-regression issue as it is observed from M60 old builds. Hence, marking it as untriaged to get more inputs from dev team. Thanks...!!
,
Sep 24
,
Sep 25
,
Sep 28
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c02251d23a5d96962aec9f0c2e92bb5909fb757c commit c02251d23a5d96962aec9f0c2e92bb5909fb757c Author: Stefan Zager <szager@chromium.org> Date: Fri Sep 28 19:57:42 2018 [IntersectionObserver] Report coordinates as CSS pixels. Prior to this patch, IntersectionObserverEntry was reporting coordinates in device pixels. BUG= 887974 Change-Id: Ic91748cd2eb9b828a787bb4372d5d5b10bcf8cfb Reviewed-on: https://chromium-review.googlesource.com/1250121 Commit-Queue: Stefan Zager <szager@chromium.org> Reviewed-by: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#595190} [modify] https://crrev.com/c02251d23a5d96962aec9f0c2e92bb5909fb757c/third_party/WebKit/LayoutTests/external/wpt/intersection-observer/bounding-box.html [modify] https://crrev.com/c02251d23a5d96962aec9f0c2e92bb5909fb757c/third_party/blink/renderer/core/intersection_observer/intersection_observation.cc [modify] https://crrev.com/c02251d23a5d96962aec9f0c2e92bb5909fb757c/third_party/blink/renderer/core/layout/intersection_geometry.cc [modify] https://crrev.com/c02251d23a5d96962aec9f0c2e92bb5909fb757c/third_party/blink/renderer/core/layout/intersection_geometry.h
,
Oct 1
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by vamshi.kommuri@chromium.org
, Sep 23