New issue
Advanced search Search tips

Issue 887025 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Themed tabstrip polish issues with dark theme

Project Member Reported by msw@chromium.org, Sep 19

Issue description

Themed tabstrip polish issues with dark theme

Hey Rob, can you elaborate the ways this theme should be rendered differently?
https://chrome.google.com/webstore/detail/marc-ecko/opjonmehjfmkejjifhhknofdnacklmjk

Perhaps tabs should be using the stroke?
Perhaps active/inactive tabs could be rendered more distinctly?

It might be nice if inactive tabs had no background at all, like the default theme...
 
themed_tab_strip_polish.png
72.7 KB View Download
Are you looking for possible Chrome bugs, or asking about ways we could update the theme?  I think the issues you raise are legit, but most of them are in the realm of "update the theme" rather than "Chrome bug".
Cc: thomasanderson@chromium.org pkasting@chromium.org
One potential change on the Chrome side may be to change the color of the stroke.

Adding pkasting@ and thomasanderson@ for this fun theme issue.
I don't have any strong opinions on Chrome's treatment here, perhaps it does come down to the theme.
I mainly filed this bug at Rob's request, feel free to morph or close it as appropriate.

I do really like how the classic theme doesn't paint an inactive tab background at all.
The simple separator definitely looks cleaner, but I guess it all depends on the theme.
inactive_tab_backgrounds.png
11.0 KB View Download
@2: For stroke colors, since the theme here isn't using a stroke, I don't know what color it would be drawn.  That said, Monica would like themes to have programmatic control over the stroke (at least color, maybe other stuff).

@3: The theme controls whether it wants an inactive tab background, so we could change the theme here to avoid those.

I think in the end this is one where we should just update the theme to look better.  Who owns these?
Labels: -Proj-MdRefresh Proj-DesktopUI
Labels: Group-Themes
Labels: Hotlist-DesktopUIConsider
Labels: M-71 Target-71
Owner: rameier@chromium.org
Status: Assigned (was: Untriaged)
Reassigning to rameier@
Labels: -Hotlist-DesktopUIConsider Hotlist-DesktopUITriaged
Labels: Hotlist-DesktopUIChecked Hotlist-DesktopUIToolingRequired
Labels: -M-71 -Target-71 M-73 Target-73
Owner: ----
Status: Available (was: Assigned)
Mass-unassigning bugs from rameier@.

Sign in to add a comment