New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 782802 link

Starred by 27 users

Issue metadata

Status: Closed
Owner:
Closed: Dec 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

ChromeOS issue: Divided horizontal scroll not appearing since v60

Project Member Reported by vkhabarov@chromium.org, Nov 8 2017

Issue description

ChromeOS version: 60-62
ChromeOS device model: any
Case#: 13712481

Description: Website has some implementation of horizontal scroll in html table, where number of columns stay pinned and others are scrolled. It works on other platforms, other browsers, but not on ChromeOS since v60 (verified on v59 - WAI).


Steps to reproduce: 
(reproduction credentials are only available from US and India)
1. Go to (url in https://docs.google.com/a/google.com/document/d/1aD1Hn1tq9lC5TkTvLyuVfQDzPCbwNYHPX2w0i3wb3DY/edit?usp=sharing). This is a set of dummy data. 
2. For user name use (username in https://docs.google.com/a/google.com/document/d/1aD1Hn1tq9lC5TkTvLyuVfQDzPCbwNYHPX2w0i3wb3DY/edit?usp=sharing), and leave the password blank. 
3. CRITICAL: Click ""Classic Login"". Do not press Enter, or you will be taken to the wrong area. 
4. Click the red gradebook icon on the period 1 GEOMETRY class line. 
5. You will see rows with students and columns with assignments. There are 8 homework assignments labeled HWK-001 to HWK-008. On a standard 11"" Chromebook only the first 6 assignments or so should be visible. 

Current Behavior / Reproduction: 
No scrollbar beneath the table, not way to browse those columns

Expected Behavior: 
Scrollbar which allows to scroll through homework columns

Drive link to logs: 
Video with v59 working as expected - 
https://drive.google.com/a/google.com/file/d/1fY6-kbRAAyBsnysTDubQkGrHE_nC_l5_/view?usp=sharing
Video with v62 not working as expected - 
https://drive.google.com/a/google.com/file/d/1NBQyxnzPZBaJFbst_FTTN-3oC713QwpG/view?usp=sharing

 
Cc: vkhabarov@google.com

Comment 3 by e...@chromium.org, Nov 9 2017

Owner: szager@chromium.org
Status: Assigned (was: Untriaged)
Cc: marcore@chromium.org
Stefan, could you please help triage this bug ?
This is reproducible on linux with the #overlay-scrollbars flag enabled.

Off the top of my head, my theory is that the hit test is getting clipped to the bounds of the height:1px div that actually owns the scrollbar.
Are there any updates to this bug which I can provide to customer?

Comment 7 by szager@chromium.org, Dec 22 2017

Status: Started (was: Assigned)
I have a fix in progress, but it's unlikely to land until the first week of January.
Project Member

Comment 8 by bugdroid1@chromium.org, Jan 5 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b541aa7c439e96b9c00ce6a70d7baa4fa8f17b48

commit b541aa7c439e96b9c00ce6a70d7baa4fa8f17b48
Author: Stefan Zager <szager@chromium.org>
Date: Fri Jan 05 22:45:29 2018

Fix hit testing overlay scrollbars

The previous code computed the wrong y-coordinate for the top of the
horizontal scrollbar.

Bug:  782802 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Ic31f94ca2a3e45881682940ae28f754894159a51
Reviewed-on: https://chromium-review.googlesource.com/843777
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Stefan Zager <szager@chromium.org>
Cr-Commit-Position: refs/heads/master@{#527415}
[modify] https://crrev.com/b541aa7c439e96b9c00ce6a70d7baa4fa8f17b48/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
[modify] https://crrev.com/b541aa7c439e96b9c00ce6a70d7baa4fa8f17b48/third_party/WebKit/Source/core/paint/PaintLayerScrollableAreaTest.cpp

Can we confirm if the fix has landed? 
it's in canary:
Commit b541aa7c... initially landed in 65.0.3313.0

We are still seeing the same issue even with the latest version of Canary.  Here is a link with a video of the behavior.

https://drive.google.com/file/d/1zPuKIrrcuVxSYfcbGX226Bdjvne-V3JB/view?usp=sharing

Labels: M-64
We have another customer reporting that fix in latest Canary doesn't resolve their problem, could you please resume work on this?
Hi all, do we have any updates on this? Thanks. 
szager@, can you please provide an update?

Comment 15 by jmo...@avr2.org, Feb 6 2018

There still appears to be no scroll bar.
Owner: bokan@chromium.org
Status: Assigned (was: Started)
This appears to be an issue in the compositor code; re-assigning to bokan@, who has the knowledge.
bokan@
Is there any updated that can be shared with the customer?
do you know any update for M64?

Comment 18 by bokan@chromium.org, Feb 16 2018

There's no updates, sorry. I'm OOO all next week so I won't be able to look into this more until at least then.
Cc: marchuk@google.com
Labels: M-65
@bokan, still not working on 
10323.62.0
65.0.3325.184
Can you please help to take a look?
I had a look at this yesterday. Unfortunately, it looks like this is just a site that interacts very poorly with the new scrollbar design.

They somehow measure scrollbar thickness on load and create a box below the table that's exactly as tall as the scrollbar thickness. They then use the scrollable content width and create an invisible box inside the "scrollbar box" to that it overflows in the same way. They hook up an onScroll listener to this box and use that to drive scrolling of various boxes in the table.

Android browsers have the same issue but the page has a specific touch listener over the right-most, scrollable columns so they can be scrolled with touch. Unfortunately, they don't have a listener for wheel events.

This would also be a problem on Mac (both Safari and Chrome) since they have similar scrollbars. However, it looks like the page has a workaround for overlay scrollbars there by applying a custom ::-webkit-scrollbar CSS style. This workaround would work just as well on ChromeOS but it appears to be a UA check specifically for Mac :(.

Is there any way to reach out to the developer of the app? The fix is extremely simple on their end - apply the same workaround on ChromeOS as on Mac.
Cc: sgabr...@chromium.org tbuck...@chromium.org
+sgabriel and tbuckley@ for PM side of scrollbars.

Comment 22 by jwag...@avr2.org, Apr 5 2018

@bokan: I will contact the developer with your comments and report back when I get a reply.  However, there is also  bug #778992 , which some of us believe is a duplicate of this one.  I don't have a direct line to that site's developers.


Thanks jwagner@.  bug 778992  is actually different. I'll take another look at it again in the next week or two to double-check the issue and see if we can do something about it in the near term.

In the mean-time, if folks have examples of other pages where scrollbars are broken and they suspect this is the same issue, please file a separate bug and assign to me - I'd like to investigate and determine what issues we're hitting in the wild.

Comment 24 by jwag...@avr2.org, Jun 27 2018

The problem still persists in 68.0.3440.34 even though Google support said that they though 68 might fix the issue.

@bokan: You were going to look at this again back in April.  Can you provide any further info?  Thanks.

Hello!
This bug is receiving this notice because there has been no acknowledgment of its existence in quite a bit of time
- If you are currently working on this bug, please provide an update.
- If you are currently affected by this bug, please update with your current symptoms and relevant logs.

If there has been no updates provided by EOD Wednesday, 12/19/18 (5pm EST), this bug will be archived and can be re-opened at any time deemed necessary.

Thank you!
Our software developer finally fixed this issue for us, and Google closed our support ticket some time ago.  As far as we are concerned this issue may be closed.  Thanks.

Status: Closed (was: Assigned)
Thank you for the update.

Sign in to add a comment