Battery Indicator is black on the panel

Bug #950264 reported by Danielle Foré
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Wingpanel
Fix Released
Medium
Danielle Foré
elementary OS
Fix Released
Critical
Unassigned

Bug Description

Battery indicator in the panel is black instead of white.

Related branches

Changed in elementaryos:
milestone: none → 0.2-luna
importance: Undecided → Medium
Changed in elementaryos:
milestone: 0.2-luna → luna-beta1
Changed in elementaryos:
importance: Medium → Critical
status: New → Confirmed
Revision history for this message
xapantu (xapantu) wrote :

This is not a problem with the icons?

Changed in wingpanel:
assignee: nobody → Daniel Fore (daniel-p-fore)
status: New → In Progress
Revision history for this message
Danielle Foré (danrabbit) wrote :

No, they are -symbolic icons.

I've proposed a branch merge that fixes the issue.

Revision history for this message
Victor Martinez (victored) wrote :

Daniel, I think that this should not be fixed in Wingpanel.

Setting the foreground color in Wingpanel's stylesheet is equivalent to hardcoding them inside the application, breaking compatibility with other themes (light text and icons under Radiance, for instance).

Now that we have symbolic icons, it will be theme's fault if they have not the same foreground color as the text.

Revision history for this message
Danielle Foré (danrabbit) wrote :

But we do ship a default theme with wingpanel that sets the text color already. The merge I've proposed just adds GtkImage to the list of classes.

There is absolutely no code whatsoever in any theme that deals with wingpanel. If you think we should move this theming into eGTK and no longer supply a theme that's fine, but that means we need to add the class names to wingpanel so that this is possible. And it guarantees breakage in other themes because no other theme cares about wingpanel at all.

Ambiance and Radiance will never be a perfect fit for Pantheon because they are Unity themes. That is just the reality of it.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Ambiance works perfectly with Wingpanel here. Screenie attached.

Revision history for this message
Victor Martinez (victored) wrote :

Daniel, I agree with you. There are some facts to take into consideration though:

1) Most themes (Ambiance and Radiance included) provide styling for the "gnome-panel-menu-bar" class. It is not taking effect in Wingpanel because the default style is loaded with a very high priority (600, equivalent to GTK_STYLE_PROVIDER_PRIORITY_APPLICATION), making it impossible to override the style from the user's theme.

2) Wingpanel also uses the "menubar" class, and since there's no style information regarding that class in the default style sheet, it can be overriden. That's why the panel looks like a menubar in Sergey's screenshot. The foreground color still follows the default style.

In conclusion, we need to fix the priority issue described in (1) and remove the style class mentioned in (2). If we do that, it's fine if we leave the foreground color in the stylesheet.

Changed in wingpanel:
status: In Progress → Fix Committed
importance: Undecided → Medium
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

The text in Wingpanel's menus is unreadable in Ambiance now.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

(it's still white in Unity, so it doesn't look like a theming issue. Also, I can't guarantee it's caused by the change described in this bug)

Changed in elementaryos:
status: Confirmed → Fix Released
status: Fix Released → Fix Committed
Changed in elementaryos:
status: Fix Committed → Fix Released
Changed in wingpanel:
milestone: none → luna-beta1
Changed in wingpanel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.