It's strange to read
You are taking my words out of context "I'm offering to ...", "Other than that".
this was slow
I don't see any delay using SVG icons.
we moved away from svg to png
It is not necessery to load SVG. I already wrote "Bitmap icons can be fine-tuned for several sizes". Have you ever looked into how icons are organized in some widely used projects?
we could have kept the icons in svg mode but then another problem, the designer that did the icons used Illustrator and .ai files are a pain to open without it
Strange, I didn't use Illustrator at all. And there already are SVG icons in Subsurface.
Source format has to be SVG, if there is any intention to let someone work on the icons and be able to scale them for HiDPI displays.
everyone was happy
Not me. Current icons are "organized" in a complete mess. Some are in theme, some are not. They are in several different formats and sizes. They are unresizable. Some icons are unreadable. Many icons make no sense - there is no way to figure out what action they are supposed to stand for. They are not using the same color palette and style. Subsurface is using it's own icons for actions widely used through the whole GUI.
Some are better, some are not
As I wrote "consider this a proposal", "I'm open to suggestions."
So I really don't understand you.
The most important things in my patches are naming, deduplication, organization. I would love to let everybody use the icons he likes. But with current codebase, it's not possible.
My time is precious. There were no constructive criticism. I don't see any way how to split this work into smaller pieces.