LMMS / lmms

Cross-platform music production software
https://lmms.io
GNU General Public License v2.0
8.13k stars 1.01k forks source link

Many file menu actions use incorrectly sized icons #5569

Open Spekular opened 4 years ago

Spekular commented 4 years ago

Bug Summary

File menu icons should be 16x16 in order to display pixel perfect. Only two of the actions in the file menu are actually at this size.

Screenshot

Screenshot of file menu, side to side by excerpt from LMMS icons file Image key:

Proposal

I think this is a symptom of poor organization of icons. Icon file names don't indicate their size, nor are they organized in subfolders by size. I think the rendered icons in the theme ought to be split into folders by size. Then we can have something like 16x16/project_save and 20x20/project_save, which would make it very clear which size is in use.

I'm unsure of the best way to organize the different sizes when it comes to the source .svg files. If I'm not mistaken, we intended to make each icon its own file? Perhaps all size variations could be kept in one file, or perhaps they should mirror the structure of the exported images.

ryuukumar commented 4 years ago

Does LMMS code default to higher res pixmaps or 16x16 pixmaps? If it's the former the orange should be an easy fix imo.

Spekular commented 4 years ago

I don't believe there's any defaulting at all, pixmaps are selected directly by their filename from what I can tell (with .png excluded)

qnebra commented 4 years ago

I looked into icons: