Merge lp:~3v1n0/ubuntu-themes/gtk-theme-colors-define into lp:ubuntu-themes
Proposed by
Marco Trevisan (Treviño)
on 2017-05-16
| Status: | Merged |
|---|---|
| Approved by: | Marco Trevisan (Treviño) on 2017-05-16 |
| Approved revision: | 546 |
| Merged at revision: | 544 |
| Proposed branch: | lp:~3v1n0/ubuntu-themes/gtk-theme-colors-define |
| Merge into: | lp:ubuntu-themes |
| Diff against target: |
190 lines (+121/-13) 2 files modified
Ambiance/gtk-3.20/gtk-main.css (+61/-6) Radiance/gtk-3.20/gtk-main.css (+60/-7) |
| To merge this branch: | bzr merge lp:~3v1n0/ubuntu-themes/gtk-theme-colors-define |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Iain Lane | 2017-05-16 | Needs Information on 2017-05-16 | |
|
Review via email:
|
|||
Commit Message
Ambiance, Radiance: define theme_* colors based on ours
Some apps customizing their CSS might use them to override
the default values, so we need to provide a definition.
Description of the Change
Example of affeted app is gitg (without defining @theme_
To post a comment you must log in.
| Daniel van Vugt (vanvugt) wrote : | # |
| Iain Lane (laney) wrote : | # |
couple of comments, feel free to land after resolving them
review:
Needs Information
| Marco Trevisan (Treviño) (3v1n0) wrote : | # |
Daniel, no... I guess we want a proper solution for that.
| Marco Trevisan (Treviño) (3v1n0) wrote : | # |
Ok, updating as Laney requested
lp:~3v1n0/ubuntu-themes/gtk-theme-colors-define
updated
on 2017-05-16
- 546. By Marco Trevisan (Treviño) on 2017-05-16
-
Ambiance, Radiance: improve wording and fix Radiance copy-paste typo

Does this cover bug 1690966?