New issue
Advanced search Search tips

Issue 871838 link

Starred by 2 users
Status: Fixed
Owner:
kylixrd@chromium.org
Closed: Aug 20
Cc:
pkasting@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Hovering multiple adjacent selected tabs doesn't paint in the correct order.

Project Member Reported by kylixrd@chromium.org, Aug 7 
Open several tabs
Multi-select at least 3 adjacent tabs
Hover mouse over middle tab
Lower tab shape "tails" should paint over the tab to the left while hovered or the hovered animation is running.

Comment 1 by kylixrd@chromium.org, Aug 7 

Added video demonstrating the incorrect effect.
HoveredSelectedTabShort.mp4
129 KB View Download

Comment 2 by kylixrd@chromium.org, Aug 7

Summary: Hovering multiple adjacent selected tabs doesn't paint in the correct order. (was: Hovering multiple adjacent selected tabs doesn't paint in the correct ordering.)
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 8 

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

commit 332bdd7bc8e88b3a1a927500d1723ed24523f1fa
Author: Allen Bauer <kylixrd@chromium.org>
Date: Wed Aug 08 01:04:39 2018

Re-order selected tabs painting for hover effects.

Hovering over selected tabs will not properly order painting such that the
painting order will paint the lower tab "tails" over the left adjacent tab.

While this has always been present, with the recent added painting of the
stroke on the tabs, this is much more apparent.

Bug:  871838 
Change-Id: I4dd4c4264107fb8a058b5210b02c27008498f35d
Reviewed-on: https://chromium-review.googlesource.com/1165689
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#581423}
[modify] https://crrev.com/332bdd7bc8e88b3a1a927500d1723ed24523f1fa/chrome/browser/ui/views/tabs/tab_strip.cc

Comment 4 by robliao@chromium.org, Aug 9

Labels: -M-69 -Target-69 M-70 Target-70

Comment 5 by robliao@chromium.org, Aug 20

Labels: -Target-70 -M-70 M-X

Comment 6 by pkasting@chromium.org, Aug 20

Labels: -M-X M-70
Status: Fixed (was: Assigned)
Should be fixed in 70

Comment 7 by robliao@chromium.org, Aug 21

Labels: Proj-DesktopUI

Comment 8 by robliao@chromium.org, Sep 13

Labels: Hotlist-MdRefreshDesignPolish
►

Sign in to add a comment