Merge lp:~jsjgruber/ubuntu/maverick/light-themes/light-themes.fix640096 into lp:ubuntu/maverick/light-themes

Proposed by John S. Gruber
Status: Merged
Merge reported by: Ken VanDine
Merged at revision: not available
Proposed branch: lp:~jsjgruber/ubuntu/maverick/light-themes/light-themes.fix640096
Merge into: lp:ubuntu/maverick/light-themes
Diff against target: 26 lines (+8/-0)
2 files modified
Ambiance/gtk-2.0/apps/gnome-panel.rc (+1/-0)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~jsjgruber/ubuntu/maverick/light-themes/light-themes.fix640096
Reviewer Review Type Date Requested Status
Robert Ancell Approve
Review via email: mp+37351@code.launchpad.net

Description of the change

Fix lp: 640096 in the Ambiance theme by including the CPUFreq applet as those that are in a panel and, therefore, styled with the panel style.

After applying test by selecting the Ambiance theme and then adding the CPU Frequency Selector to the top panel. The frequency at which the computer's CPU is running should be visible. (If not, right click the added widget and check its preferences to make sure Display Settings Appearance: includes the Text setting).

Without the change the widget text displaying the frequency text is rendered so close in color to that background that its effectively invisible irrespective of widget preferences.

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Confirmed fix works

review: Approve
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Actually merged into lp:light-themes, it will sync back to the source package branch

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Ambiance/gtk-2.0/apps/gnome-panel.rc'
2--- Ambiance/gtk-2.0/apps/gnome-panel.rc 2010-08-26 14:06:05 +0000
3+++ Ambiance/gtk-2.0/apps/gnome-panel.rc 2010-10-02 14:51:44 +0000
4@@ -106,6 +106,7 @@
5 widget "*PanelWidget*" style "panel"
6 widget "*PanelApplet*" style "panel"
7 widget "*fast-user-switch*" style "panel" # workaround for Fast User Switch applet
8+widget "*CPUFreqApplet*" style "panel" # workaround for CpuFreq Applet
9 class "PanelApp*" style "panel"
10 class "PanelToplevel*" style "panel"
11 #widget_class "*Mail*" style "panel"
12
13=== modified file 'debian/changelog'
14--- debian/changelog 2010-09-27 16:42:10 +0000
15+++ debian/changelog 2010-10-02 14:51:44 +0000
16@@ -1,3 +1,10 @@
17+light-themes (0.1.8.1ppa3) maverick; urgency=low
18+
19+ * Include cpu-freq among the panel applets known to the Ambiance theme
20+ (LP: #640096)
21+
22+ -- John S Gruber <JohnSGruber@gmail.com> Fri, 01 Oct 2010 23:00:56 -0400
23+
24 light-themes (0.1.8.1) maverick; urgency=low
25
26 * helps making disabled menu items more readable (LP: #617192)

Subscribers

People subscribed via source and target branches