Thanks for geany.
Could you please not hardcode the window icon? Many icon themes have a 'geany' icon nowadays and currently they are only used on the menu because the window icon is hardcoded.
Not a bug, moved to Feature request. Many themes may set the icon, but not all do, so that case would have to be properly handled on all platforms not simply do not set any icon. Patches are welcome.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hi, that's not an issue.
geany ships its own icons for the hicolor icon theme, which is the default fallback theme. If an icon theme doesn't have a geany icon it automatically uses those.
So I'm just asking that it doesn't call a specific icon with the full path which is what I'm guessing it does and instead simply call the 'geany' icon.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
In that case the problem was that it was using 'yumex.png' instead of simply 'yumex'. With that change it now can use the chosen icon theme if it has a yumex icon, otherwise it falls back to the bundled icon.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Not a bug, moved to Feature request. Many themes may set the icon, but not all do, so that case would have to be properly handled on all platforms not simply do not set any icon. Patches are welcome.
Hi, that's not an issue.
geany ships its own icons for the hicolor icon theme, which is the default fallback theme. If an icon theme doesn't have a geany icon it automatically uses those.
So I'm just asking that it doesn't call a specific icon with the full path which is what I'm guessing it does and instead simply call the 'geany' icon.
For example, yumex (fedora package manager) had this issue and it was solved by this simple commit:
https://github.com/timlau/yumex/commit/65d349a886742c103cfc7bf230bae76589d69043
In that case the problem was that it was using 'yumex.png' instead of simply 'yumex'. With that change it now can use the chosen icon theme if it has a yumex icon, otherwise it falls back to the bundled icon.
Sorry, the commits are
https://github.com/timlau/yumex/commit/d739c61faa428b9bc06b6f1a434cd948be4f78a7
and
https://github.com/timlau/yumex/commit/65d349a886742c103cfc7bf230bae76589d69043
Now implemented in Git.