Open Bug 1347188 Opened 7 years ago Updated 1 year ago

Add Google Chrome tab background properties

Categories

(WebExtensions :: Themes, enhancement, P5)

Product:
WebExtensions
Component:
Themes
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: mikedeboer, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: triaged)

User Story

Implement:

tints.background_tab

as part of chrome parity.
See (again) https://docs.google.com/spreadsheets/d/1YScpOVL5WdNDhQY2Nngh4YkK0bOpkfwJvpRjpMSxMWo/edit#gid=0

Constants as defined in https://chromium.googlesource.com/chromium/src/+/master/chrome/browser/themes/theme_properties.h are:
 * COLOR_TAB_TEXT,
 * COLOR_BACKGROUND_TAB_TEXT,
 * TINT_BACKGROUND_TAB,
 * COLOR_BACKGROUND_TAB,
 * COLOR_TAB_THROBBER_SPINNING,
 * COLOR_TAB_THROBBER_WAITING,

It's important here to first implement the properties in the order and with names that fit with Firefox well and try to match these up with the Google equivalents above.

The 'throbber' properties are used by Google Chrome to apply an alternative color to the loading spinner that appears in place of a tabs' favicon. Implementing these may not be straightforward at the moment, thus may be moved forward to a separate bug.
Severity: normal → enhancement
Priority: -- → P5
 * COLOR_TAB_TEXT -> can be implemented
 * COLOR_BACKGROUND_TAB_TEXT -> currently implemented as tab_text (which should be changed)
 * TINT_BACKGROUND_TAB -> what is the difference with colors.background_tab ?
 * COLOR_BACKGROUND_TAB -> is it relevant with photon ?
 * COLOR_TAB_THROBBER_SPINNING -> --tab-loading-fill
 * COLOR_TAB_THROBBER_WAITING -> can be implemented
Depends on: 1415872
No longer depends on: 1415872
Summary: Add Google Chrome tab color properties → Add Google Chrome tab background and spinner color properties
>  * COLOR_TAB_TEXT -> can be implemented
fixed
>  * COLOR_BACKGROUND_TAB_TEXT -> currently implemented as tab_text (which
> should be changed)
fixed

>  * TINT_BACKGROUND_TAB -> what is the difference with colors.background_tab ?
>  * COLOR_BACKGROUND_TAB -> is it relevant with photon ?
left for this bug

>  * COLOR_TAB_THROBBER_SPINNING -> --tab-loading-fill
>  * COLOR_TAB_THROBBER_WAITING -> can be implemented
I've separated this out to bug 1426686.
Depends on: 1426686
User Story: (updated)
No longer depends on: 1426686
Summary: Add Google Chrome tab background and spinner color properties → Add Google Chrome tab background properties
mass move of existing themes bugs to new WebExtensions: Themes component
Component: WebExtensions: Frontend → WebExtensions: Themes
It looks like Chrome only supports tints.background_tab. Removing colors.background_tab from user story.
User Story: (updated)
Product: Toolkit → WebExtensions
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.