Merge lp://staging/~eye-zak-devel/light-themes/fixes-lp954437 into lp://staging/~ubuntu-art-pkg/light-themes/trunk

Proposed by z
Status: Needs review
Proposed branch: lp://staging/~eye-zak-devel/light-themes/fixes-lp954437
Merge into: lp://staging/~ubuntu-art-pkg/light-themes/trunk
Diff against target: 184 lines (+55/-14)
2 files modified
Ambiance/gtk-3.0/apps/gnome-panel.css (+26/-7)
Radiance/gtk-3.0/apps/gnome-panel.css (+29/-7)
To merge this branch: bzr merge lp://staging/~eye-zak-devel/light-themes/fixes-lp954437
Reviewer Review Type Date Requested Status
Light Themes Contributors Pending
Review via email: mp+97295@code.staging.launchpad.net

Description of the change

Fixes lp:954437

This bug affects only gnome-fallback sessions

Show desktop and tasklist have light backgrounds, they should be dark like the rest of the panel.

Font colour in main menu panel menus is grey, not white.

To post a comment you must log in.
219. By z

Fix main menu css selection to be more specific

Revision history for this message
Benjamin Drung (bdrung) wrote :

Thanks for your work. Your proposed branch fixed bug #955376 and #981289.

The main menu icon is smaller than it used to be in GNOME 2. Can you make it bigger?

Revision history for this message
Jeremy Bícha (jbicha) wrote :

Benjamin, the smaller menu icon is an upstream bug (which hasn't been filed yet). It would be nice if it were the same size as in the "main menu" applet. Because it's so small and barely distinguishable, I think we should go ahead and leave the menu bar icon disabled for Precise.

Unmerged revisions

219. By z

Fix main menu css selection to be more specific

218. By z

Fixes lp:954437

Fix font color in main menu panel menus (Ambiance and Radiance)
Include icon in main menu (Ambiance and Radiance)
 - this is debatable, I am just making it more like gnome2

Make "show desktop" and tasklist behave like theme suggests (Ambiance and Radiance)
 - I tried to do this without customizing the behavior the way I thought best. What it should be is debatable, maybe someone with knowledge about this can change it.

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.