Merge lp://staging/~vanvugt/ubuntu-themes/fix-1725921 into lp://staging/ubuntu-themes

Proposed by Daniel van Vugt
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: 601
Merged at revision: 601
Proposed branch: lp://staging/~vanvugt/ubuntu-themes/fix-1725921
Merge into: lp://staging/ubuntu-themes
Diff against target: 28 lines (+0/-11)
1 file modified
Ambiance/gtk-2.0/gtkrc (+0/-11)
To merge this branch: bzr merge lp://staging/~vanvugt/ubuntu-themes/fix-1725921
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Review via email: mp+340265@code.staging.launchpad.net

Commit message

Remove an old workaround which would force combobox menu text
to be dark. Starting in r583 the combobox menu background also
changed to dark, so forcing the text to dark made it almost the
same colour as the background. (LP: #1725921)

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Looks good, thanks.

Can you also prepare a MP tracking X for backporting to xenial?

review: Approve
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Yes, certainly I will do xenial. But correct process is to make sure bionic is done first.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches