battery-*-symbolic icon not properly rendered in the panel

Bug #845037 reported by Sam Tate
172
This bug affects 41 people
Affects Status Importance Assigned to Milestone
elementary Icons
Invalid
Undecided
Unassigned
indicator-power
Fix Released
Medium
Lars Karlitski
indicator-power (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Although indicator-power seems to use battery-*-symbolic icons, it stretches the icon out and/or crops it instead of using it at the intended 16px height. Creating a 22px -symbolic icon results in it still being stretched outside the drawn area making it not possible to correctly supply a -symbolic icon here.

Related branches

Revision history for this message
Cassidy James Blaede (cassidyjames) wrote :

Speaking of the power indicator, we really need to decide on https://blueprints.launchpad.net/elementaryos/+spec/indicator-strategy.

That may affect this, especially if we're remaking an indicator or something.

Revision history for this message
Paweł Pacana (pawelpacana) wrote :

This is how battery renders in my indicator-power: http://yfrog.com/njzaznaczenie004p

Revision history for this message
Paweł Pacana (pawelpacana) wrote :

Another screenshot for completness: http://img252.imageshack.us/img252/5458/zaznaczenie007.png

This is Ubuntu 11.10 with classic Gnome session and elementary-icon-theme 3.0-0~r906+pkg19~oneiric1.

Revision history for this message
Scott Ringwelski (sgringwe) wrote :

I have the same problem using revision 906.

http://i.imgur.com/vLAoz.png

Revision history for this message
Avi Romanoff (aroman) wrote :

I am affected by this too, and I recall discussing it with Dan. Something about GNOME 3 not using proper symbolic icons or something. Might be something we want to explicitly edge-case?

Marking as confirmed -- feel free to change it.

Changed in elementaryicons:
status: New → Confirmed
Revision history for this message
Danielle Foré (danrabbit) wrote :

This is partially an elementary bug because I'm currently borrowing GNOME's battery icons, but it's mostly an indicator-power bug because it's stretching and/or cropping the icon for some unknown reason.

summary: - The elementary battery icon in Ubuntu 11.10 doesn't fit in
+ battery-*-symbolic icon not properly rendered in the panel
description: updated
description: updated
Changed in indicator-power:
status: New → Confirmed
importance: Undecided → Medium
Changed in indicator-power (Ubuntu):
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Lucazade (lucazade) wrote :

is this bug going to be fixed??!

Revision history for this message
Brendan William (bwilliam) wrote :

This bug has now been fixed with the latest elementary icon set (ppa:elementary-os/daily)

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

No this bug has not been fixed. It has been worked around by not using symbolic battery icons in elementary. The problem still persists.

Revision history for this message
Brendan William (bwilliam) wrote :

I misunderstood the bug XD. indicator-power stretches out wingpael because of this, taking up space.

Revision history for this message
Lars Karlitski (larsu) wrote :

indicator-power requests icons of size GTK_ICON_SIZE_LARGE_TOOLBAR instead of GTK_ICON_SIZE_MENU. If I change this, the gnome icon theme looks correct, but the battery icons from ubuntu-mono-dark become tiny.

Daniel, you seem to be the one who last touched the battery icons in ubuntu-mono-dark. Do you know what I need to change in there to make the icons appear at the right size? I'm hesitant to include the fix to indicator-power in precise if it breaks the default icon set.

Revision history for this message
Lars Karlitski (larsu) wrote :

Cimi figured it out: the svg files in ubuntu-mono-* contain a padding around the icons that is too large.

My indicator-power patch also isn't correct: the panel uses a different icon size alltogether, which is pulled from libindicator. Libindicator doesn't support multiple icon name backups, which are needed by the power panel (it looks for all sorts of icons).

Together with the changes necessary for the icons, this is just too invasive a change for this late in the cycle. So, this is not going to be fixed in precise, but shortly after. Sorry.

Changed in indicator-power:
assignee: nobody → Lars Uebernickel (larsu)
Revision history for this message
Charles Kerr (charlesk) wrote :

larsu, is this still an issue in >= 12.10?

Revision history for this message
982c80311320c1b (alexander-wilms) wrote :

It is, at least for me.

Revision history for this message
Lars Karlitski (larsu) wrote :

Charles, yes it is.

Revision history for this message
Atom Stark (atomstark) wrote :

Still unsolved after 2 years ...

Revision history for this message
Francisco Cribari (cribari) wrote :

Stil lunresolved?

Revision history for this message
Adolfo Jayme Barrientos (fitojb) wrote :

This looks fixed for me, have you tested 14.04?

Changed in indicator-power (Ubuntu):
status: Confirmed → Incomplete
Changed in elementaryicons:
status: Confirmed → Invalid
Revision history for this message
Jan Schürmann (visionfactory.net) wrote :

seems to be fixed in 14.04

Changed in indicator-power:
status: Confirmed → Fix Released
Changed in indicator-power (Ubuntu):
status: Incomplete → 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.