wrong icon for the power indicator after last 12.04 update

[mlaverdiere]

I'm on 12.04. Power-indicator icon in the upper/menu bar was fine until yesterday's update. Now, the icon (when battery is fully charged & AC connected) doesn't match the regular Ubuntu/unity/Ayatana indicators icon style (like if it was the gnome-shell style used instead). I'll try to put a screenshot later, if needed (I'm not on an Ubuntu machine right now).

[Ken VanDine (ken-vandine)]

Reassigning to gtk+3.0, this was introduced with gtk 3.4.2 which included a change for finding symbolic icons.

http://git.gnome.org/browse/gtk+/commit/gtk/gtkicontheme.c?id=271adacfadfdd81b27fb12356fe20cde3094de37

I have reproduced this with 3.4.2-0ubuntu0.1 in precise-proposed and 3.4.2-0ubuntu1 in quantal

[Stéphane Guillou (stephane-guillou)]

I confirm this on my Samsung N310 with 12.04 up to date. Here is a screenshot.

[Launchpad Janitor (janitor)]

This bug was fixed in the package gtk+3.0 - 3.4.2-0ubuntu2

---------------
gtk+3.0 (3.4.2-0ubuntu2) quantal; urgency=low

  * debian/patches/101_revert_symbolic_icon_search.patch
    - Revert symbolic icon search changes from 3.4.2 (LP: #1001229)
 -- Ken VanDine <email address hidden> Fri, 18 May 2012 10:01:55 -0400

[Ken VanDine (ken-vandine)]

Added an indicator-power task to make sure we aren't doing something wrong with the symbolic icons.

[Lars Karlitski (larsu)]

These icon names are coming from upower for a discharging battery:

  battery-full-symbolic gpm-battery-100 battery-full

Gtk looks for the first icon name in the current theme. If it doesn't find it, it proceeds to the next icon in the list, etc. Since ubuntu-mono doesn't have battery-full-symbolic, it advanced to gpm-battery-100, which it does have. If none of the icons in the list are found in the current theme, gtk proceeds to look for them in the parent icon theme.

The linked commit changes gtk to treat the -symbolic suffix a little differently, because it really isn't about specificity, but a visual property of the icon. So it looks for those in the parent theme *first*. Parent theme for ubuntu-mono seems to be the default gnome set, which contains a colorful rendition of battery-full-symbolic.

The real fix for this is to agree upon a naming scheme for symbolic icons. I suggest for ubuntu-mono to simply drop the old names and adapt the -symbolic ones.

In addition, indicator-power needs to look for the -symbolic icons first. I assume that it can even drop all the other names, as noone else is using them afaik.

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Martin Pitt (pitti)]

Hello mlaverdiere, or anyone else affected,

Accepted gtk+3.0 into precise-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

[Sebastien Bacher (seb128)]

Ken, thanks for uploaded a fixed version! I'm not sure we should revert that patch in q though, why not fixing the indicators there?

[Haw Loeung (hloeung)]

Hi Martin Pitt,

Updated the following packages and it seemed to have fixed it:

Upgrade: gir1.2-gtk-3.0:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2), libgtk-3-bin:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2), libgtk-3-0:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2), libgail-3-0:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2), libgtk-3-common:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2)

Thanks,

Haw

[Dario Zanotto (d.zanotto)]

Hello all,
After updating the folowing packages, it seemed to be fixed:

gir1.2-gtk-3.0:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2)
libgtk-3-bin:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2)
libgtk-3-0:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2)
libgail-3-0:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2)
libgtk-3-common:amd64 (3.4.2-0ubuntu0.1, 3.4.2-0ubuntu0.2)

Thank you,
Dario

[Launchpad Janitor (janitor)]

This bug was fixed in the package gtk+3.0 - 3.4.2-0ubuntu0.2

---------------
gtk+3.0 (3.4.2-0ubuntu0.2) precise-proposed; urgency=low

  * debian/patches/101_revert_symbolic_icon_search.patch
    - Revert symbolic icon search changes from 3.4.2, it breaks the icon
      displayed in indicator-power because of the order the icons are searched
      in. It looks like ubuntu-mono should drop the old icon names and
      use the new -symbolic icons. We need to carry this patch until
      ubuntu-mono fixes it's handling of symbolic icons. (LP: #1001229)

gtk+3.0 (3.4.2-0ubuntu0.1) precise-proposed; urgency=low

  * New upstream version
    - Evince 3.4: adding new annotation is not possible (lp: #984878)
    - "Delete confirmation" dialog has wrong padding (lp: #981375)
  * git patches backported
  * git_touchpad_no_leave_event.patch:
    - "gdk: do not send leave event on touchpad button release"
  * git_filechooser_wm_role.patch:
    - "Set the wm_role on GtkFileChooserDialog"
  * git_set_orientation.patch:
    - "box: set orientation style classes when setting orientation"
  * git_dnd_lock.patch:
    - "GDK X11 DND: Fix infinite loop" (lp: #880962)

  [ Jeremy Bicha ]
  * settings.ini:
    Set gnome-panel menu-bar icon size to 24 so that Ubuntu logo
    for the Applications menu isn't set too small (LP: #986969)
 -- Ken VanDine <email address hidden> Fri, 18 May 2012 13:38:04 -0400

[Stéphane Guillou (stephane-guillou)]

Fixed here too. Cheers!

[Tim Lunn (darkxst)]

The removal of this upstream patch will break symbolic icons in the latest gnome-shell (3.6 beta) once it gets uploaded to Quantal.

[Tim Lunn (darkxst)]

The correct fix for this issue is for ubuntu mono to provide battery icons with a -symbolic postfix. You can then included the upstream patch with out any issues! And really that patch is absolutely required for gnome-shell 3.6.

[Sebastien Bacher (seb128)]

Right, we should drop that revert hack, I've done that in the Vcs and it will be in quantal with the next upload, we are looking at fixing the icon theme as well

[Charles Kerr (charlesk)]

These are the -symbolic icons referenced by i-power.

 "ac-adapter-symbolic"
 "gpm-monitor-symbolic"
 "battery-empty-symbolic"
 "battery-full-charged-symbolic" (and if not found, we also try"battery-full-charging-symbolic" as a fallback)
 "battery-caution-charging-symbolic"
 "battery-full-symbolic"
 "battery-good-symbolic"
 "battery-low-symbolic"
 "battery-good-symbolic"
 "battery-caution-symbolic"
 "battery-missing-symbolic"

[John Swing (john-swing)]

I'm in Quantal and i've got the same weird battery indicator since the last update.

[Charles Kerr (charlesk)]

This doesn't appear to need any code-level changes, so I'm closing the i-power component of this ticket.

[Steve Langasek (vorlon)]

The Precise Pangolin has reached end of life, so this bug will not be fixed for that release