FS#40929 - [volumeicon] Volumeicon show wrong icon in panel.

Attached to Project: Community Packages
Opened by saptech (saptech) - Saturday, 21 June 2014, 20:32 GMT
Last edited by Alexander F. Rødseth (xyproto) - Thursday, 03 July 2014, 23:39 GMT
Task Type Bug Report
Category Upstream Bugs
Status Closed
Assigned To Alexander F. Rødseth (xyproto)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:
I'm using Openbox window manager, with Tint2 panel. I have Volumeicon start from the Autostart configuration. It doesn't show the correct icon in tint2 panel. Instead of a speaker type icon, it's showing a generic square shape icon.

I was informed on IRC, Openbox channel the below conversation:

"<Narrat> Then it seems to be a problem with the volumeicon package
<Narrat> the icon is missing since the last build
<Narrat> where they switched to a specific build
<Narrat> err git commit. And with the package before there was an icon"

Additional info:
Typing 'pacman -Qi volumeicon', I left out titles that didn't include any information.

Name : volumeicon
Version : 0.5.0-3
Description : Volume control for your system tray
Architecture : i686
URL : http://softwarebakery.com/maato/volumeicon.html
Licenses : GPL3
Depends On : gtk3 alsa-lib libnotify
Installed Size : 231.00 KiB
Packager : Alexander R
Build Date : Fri Jun 13 16:45:44 2014
Install Date : Sat Jun 14 07:38:05 2014
Install Reason : Explicitly installed
Install Script : No
Validated By : Signature

Steps to reproduce:
Running Openbox with Tint2 panel and install and use Voluemicon will produce the same results.
This task depends upon

Closed by  Alexander F. Rødseth (xyproto)
Thursday, 03 July 2014, 23:39 GMT
Reason for closing:  Fixed
Comment by Alexander F. Rødseth (xyproto) - Sunday, 22 June 2014, 19:43 GMT
Thanks for reporting. You can change the icon theme in the volumeicon preferences. Does this solve the problem for you?
Comment by saptech (saptech) - Monday, 23 June 2014, 00:05 GMT
Yes, changing the icon in Preferences does work. Sorry I didn't think about checking Pref. I guess it uses the Default icon option which show the generic square icon, after installing Volumeicon.

Thanks and sorry for the inconvenience.
Comment by John Lindgren (jlindgren) - Monday, 23 June 2014, 00:25 GMT
This commit seems to be the problem:
https://github.com/Maato/volumeicon/commit/a2a12efcf8c263e5615155ef2d47bb2aff796f88

At least the Gnome icon theme as shipped by Arch doesn't have the "-panel" icon variants. It would be nice to be able to use the regular icons (from the system theme) as before. There isn't such an option in the preferences.
Comment by Alexander F. Rødseth (xyproto) - Tuesday, 24 June 2014, 09:54 GMT
Please report the issue upstream here https://github.com/Maato/volumeicon/issues so we can hear what they have to say about the issue. Thanks.
Comment by John Lindgren (jlindgren) - Tuesday, 24 June 2014, 22:24 GMT
https://github.com/Maato/volumeicon/issues/33
Comment by Steven Honeyman (stevenhoneyman) - Tuesday, 24 June 2014, 23:29 GMT
"Maato" seems to be unavailable... but this patch should fix it. If you don't have panel icons in the current theme, it uses the plain named ones instead.

I've added it to the pull request on GitHub along with the 100% cpu usage fix ( see  FS#40966  )
   0002-only-use-gtk-panel-icons... (2 KiB)
Comment by Alexander F. Rødseth (xyproto) - Saturday, 28 June 2014, 09:16 GMT
Thanks, I'll apply the patch. Would you mind forking volumeicon and also apply the patch from https://bugs.archlinux.org/task/40966, so that the fork could be used instead of the original volumeicon sources?
Comment by Alexander F. Rødseth (xyproto) - Saturday, 28 June 2014, 09:18 GMT
Or perhaps the nkoep fork would work, if issues were reported there? https://github.com/nkoep/volumeicon
Comment by Steven Honeyman (stevenhoneyman) - Saturday, 28 June 2014, 09:21 GMT
Sure, no problem. I'll fork the latest and apply just the two fixes I submitted.

nkoep has been developing some new features / code changes (updating to newer Gtk components etc) so his fork may become unstable?

Comment by Steven Honeyman (stevenhoneyman) - Saturday, 28 June 2014, 09:22 GMT
Oh, I already did :P https://github.com/stevenhoneyman/volumeicon
I never got around to deleting it!
Comment by Alexander F. Rødseth (xyproto) - Thursday, 03 July 2014, 23:25 GMT
Great, thanks. I'll update the package to use the stevenhoneyman/volumeicon repo instead.
Comment by Alexander F. Rødseth (xyproto) - Thursday, 03 July 2014, 23:39 GMT
Updated the package to use the stevenhoneyman/volumeicon repo. The updated package will appear in [community] shortly. Please re-open this bug if there are still problems with this.

Loading...