Missing menu button - Githubissues

Manuel-Kehl / Go-For-It

A stylish to-do list with built-in productivity timer. NOTE: all contributions should go to Jonathan Moerman's fork because this is where active development is taking place: https://github.com/JMoerman/Go-For-It

http://manuel-kehl.de/projects/go-for-it/

GNU General Public License v3.0

516 stars 51 forks source link

Missing menu button #14

Closed damianavila closed 9 years ago

damianavila commented 9 years ago

It seems like in other desktop envs the config button is missing... I can click on the "hidden", so it works... the problem is the "invisibility".

f931a482-8d3f-11e4-9c14-bd50d428cb65

damianavila commented 9 years ago

I am using LXDE under ubuntu 14.04...

Manuel-Kehl commented 9 years ago

@damianavila thanks for reporting! Go For It looks for icons named "open-menu" and "open-menu-symbolic" and if none of them is provided by your icon theme, "go-for-it-open-menu-fallback" is used.

Does "usr/share/icons/hicolor/24x24/actions/go-for-it-open-menu-fallback.svg" exist on your system? It should get copied there upon installation. If it does exist, then the bug may be related to the ToolBar/ToolButton widgets itself.

damianavila commented 9 years ago

The file exists...

[damian@damian-S400CA][~]$ ls -la /usr/share/icons/hicolor/24x24/actions/
total 56
drwxr-xr-x  2 root root  4096 dic 26 19:07 .
drwxr-xr-x 15 root root  4096 jul 22 19:03 ..
-rw-r--r--  1 root root   797 ene 14  2014 browse-webcam-effects.png
-rw-r--r--  1 root root  1188 ene 14  2014 cheese-take-burst.png
-rw-r--r--  1 root root  1105 ene 14  2014 cheese-take-photo.png
-rw-r--r--  1 root root 14695 dic 24 19:03 go-for-it-open-menu-fallback.svg
-rw-r--r--  1 root root   802 abr 10  2014 system-log-out.png
-rw-r--r--  1 root root  1251 abr 10  2014 system-restart.png
-rw-r--r--  1 root root  1226 abr 10  2014 system-shutdown.png
-rw-r--r--  1 root root   707 jun 17  2014 xfpm-hibernate.png
-rw-r--r--  1 root root   759 jun 17  2014 xfpm-suspend.png

Manuel-Kehl commented 9 years ago

All right. Thank you for the information! I am going to set up a few testing rigs soon. Unfortunately January will be a very busy month for me (exams, papers and presentations for university...), so I have to see which issues I prioritize. Everyone is of course invited to have a look at it and contribute fixes :wink:

damianavila commented 9 years ago

No problem... I know about priorities... when I have some time, I will try to learn some Vala and help with this... :wink:

Manuel-Kehl commented 9 years ago

@damianavila that would be awesome! If you are experienced in either Java or C# I can really recommend the following:

https://wiki.gnome.org/Projects/Vala/ValaForCSharpProgrammers https://wiki.gnome.org/Projects/Vala/ValaForJavaProgrammers

Manuel-Kehl commented 9 years ago

@damianavila I investigated into it and hopefully fixed the problem. Would you be so kind and test the new version? The fix should be in the PPA as soon as the Launchpad build-bot finishes. Simply make sure, that you have >= go-for-it-1.1-0~60 installed.

Please close the issue, if my changes have fixed the problem on your system :wink:

damianavila commented 9 years ago

Thank you sir for being so responsive... I can confirm the fix is working. And thanks for the links, I will take a look. Closing now... Cheers.