-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Some icons are missing/incorrect with icewm-menu-fdo #13
Comments
I made a diff showing what I had to change to get my icons fully working. 2c2
< menu "Audio" folder {
---
> menu "Audio" applications-multimedia {
15c15
< menu "AudioVideo" folder {
---
> menu "AudioVideo" applications-multimedia {
42c42
< menu "Development" folder {
---
> menu "Development" applications-engineering {
63c63
< menu "Game" folder {
---
> menu "Game" applications-games {
92c92
< menu "Graphics" gnome-graphics {
---
> menu "Graphics" applications-graphics {
100c100
< menu "Network" folder {
---
> menu "Network" preferences-system-network {
122c122
< menu "Office" gnome-applications {
---
> menu "Office" applications-office {
141c141
< menu "Other" gnome-other {
---
> menu "Other" applications-other {
144c144
< menu "Science" folder {
---
> menu "Science" applications-science {
149c149
< menu "Settings" folder {
---
> menu "Settings" preferences-desktop {
157c157
< menu "System" preferences-other {
---
> menu "System" preferences-system {
173c173
< menu "Utility" folder {
---
> menu "Utility" applications-utilities {
194c194
< menu "Video" folder {
---
> menu "Video" applications-multimedia {
|
@gijsbers I'd recon, his problems have following origins:
I intend to modify to icon folder iteration code, throwing glob() at it. Might increase the cache memory usage bit, see $ ls /usr/share/icons// -d If you have a better idea, please let me know. edit: s/glob/wordexp/ since it is already there. |
@Brottweiler Please fix the description or otherwise explain which parts of the standard are not followed. Here is the link: https://specifications.freedesktop.org/menu-spec/latest/index.html I do not see such icon names prescribed there. Instead, there is resolution of menu metadata via /usr/share/desktop-directories and related contents. If you don't install meta data for the menu description then there is no correct source of information we could rely on. We could, of course, hardcode a few values like it is probably done here and there (see above, "Utilities"...) but that is NOT the standard. |
@Code7R My bad, I hope the title is better now? I was following this page: https://specifications.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html for the icon naming.
Was not aware of this, how do I install the meta data for the menu description? |
Extra trickery to consider the filename of the menu description file or some equivalent category from KDE which might donate their icon and translation. For ice-wm/icewm#13
Well, you check your distribution, search for stuff which adds metadata to /usr/share/desktop-directories , or you can add similar files into your local XDG folders. You can test it but keep in mind that replacing icewm-menu-fdo is not enough, the path resolutions happens in icewm which also needs to be updated. Or check this, it's slightly misordered after grep&sort, but you get the idea.
|
Hmm, I suppose this is the fault of the icon theme then.. I installed a packages that populated |
While trying to troubleshoot why my menu is missing some icons, I noticed that the reason for that is
icewm-menu-fdo
does not use the XDG icon names I expected.For example, these three:
The above resulted in missing icons. I can fix this manually by doing the following:
Also, I changed
menu "System" preferences-other
to useapplications-system
as icon instead, since that was missing as well.I also changed
menu "Game" folder {
to useapplications-games
.Is there a good way for me to customize these icons without making my own
programs
file? I like howicewm-menu-fdo
is dynamic. Otherwise I would have to update myprograms
file every now and then.The text was updated successfully, but these errors were encountered: