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

Issue 702744 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Scrollbar visual glitch in system menu

Project Member Reported by sgabr...@chromium.org, Mar 17 2017

Issue description

How to repro:
Open system menu > Go to Network/bluetooth > Hover the scrollbar > Notice weird pixel on top and bottom of the scrollbar.

See attached.
 
zoom.png
53.2 KB View Download
Labels: Proj-MaterialDesign-CrOS
Sebastien what Chrome version did you notice this on?
59.0.3041.0

Comment 3 by est...@chromium.org, Mar 21 2017

Status: Started (was: Assigned)
woohoo I love one liners https://codereview.chromium.org/2766773002
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 21 2017

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

commit c6359868104299cc1eb5fe2700a792d52ecaac6c
Author: estade <estade@chromium.org>
Date: Tue Mar 21 18:53:33 2017

Fix Views overlay scrollbar border stroke.

Using a square end cap instead of "butt" makes the stroke extend by a
half stroke width (0.5dp or 1px @2x) at the ends.

BUG= 702744 

Review-Url: https://codereview.chromium.org/2766773002
Cr-Commit-Position: refs/heads/master@{#458493}

[modify] https://crrev.com/c6359868104299cc1eb5fe2700a792d52ecaac6c/ui/views/controls/scrollbar/overlay_scroll_bar.cc

Comment 5 by est...@chromium.org, Mar 21 2017

Status: Fixed (was: Started)

Comment 6 by est...@chromium.org, Mar 22 2017

Labels: Merge-Request-58 M-58
Status: Started (was: Fixed)
Project Member

Comment 7 by sheriffbot@chromium.org, Mar 22 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 8 by bugdroid1@chromium.org, Mar 22 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/277ab19519d29af451139a650fae93da179cc2a0

commit 277ab19519d29af451139a650fae93da179cc2a0
Author: Evan Stade <estade@chromium.org>
Date: Wed Mar 22 22:46:17 2017

Fix Views overlay scrollbar border stroke.

Using a square end cap instead of "butt" makes the stroke extend by a
half stroke width (0.5dp or 1px @2x) at the ends.

BUG= 702744 

Review-Url: https://codereview.chromium.org/2766773002
Cr-Commit-Position: refs/heads/master@{#458493}
(cherry picked from commit c6359868104299cc1eb5fe2700a792d52ecaac6c)

Review-Url: https://codereview.chromium.org/2767283002 .
Cr-Commit-Position: refs/branch-heads/3029@{#373}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/277ab19519d29af451139a650fae93da179cc2a0/ui/views/controls/scrollbar/overlay_scroll_bar.cc

Comment 9 by est...@chromium.org, Mar 23 2017

Status: Fixed (was: Started)
Labels: bluetooth
Status: Assigned (was: Fixed)
There is still some gap between the progress bar line and the top of scrollbar.

Attached screenshot from TOT 9534.0.0, 60.0.3092.0 elm

Reopening bug. Please close if this is WAI.
Screenshot 2017-05-09 at 12.15.56.png
9.5 KB View Download
Status: Fixed (was: Assigned)
I think that's because the scrollbar thumb has a white outline.
Status: Verified (was: Fixed)
Thanks estade@

9534.0.0, 60.0.3092.0

Sign in to add a comment