MenTaLguY wrote:
icons.svg has two important properties:
- it allows users to edit icon sets directly in inkscape without a
separate bulk export step (so it is important for users)
It is specific for Inkscape icons sets - it would be better if there was some kind of "export icon theme" option, that way authors of desktop icon sets (which must be in the discrete format) would also benefit.
As for Ed's opinion that icon sets should be in one SVG file: if used for the desktop, it would seriously degrade the performance, because an application that needs 5 icons would still need to load the SVG for every icon in the theme. With a single application this is less of an issue, but it can still cause a noticeable lag. We could already see this with the monolithic Tango icon theme, which contained a lot of unused variant icons and caused a big delay on startup (several seconds).
- most rendering regressions are discovered when SVG icons render
improperly in the UI (so it is important for testing)
Didn't think about this. But I suppose the rendering test suite from last year's GSoC has a greater potential to catch regressions.
That said, I won't commit this unless I come up with a way to support icons.svg too
Regards, Krzysztof Kosiński