Now that's an improvement over the previous implementation :-) That's because I didn't have time to "slice" the Tango icons yet. Another way would be to extract them automatically but I'd have to define a mapping of old IDs to new names and fix Inkscape's command line SVG export, because right now it's rather broken. But fixing it right by defining a common set of operations that all output extensions should support (e.g. "hide everything except those IDs", "crop to drawing", etc.) and implementing them is a large task. This would be best done an the SVG level by preprocessing the SVG document given to the output/print extensions.
Alexandre Prokoudine wrote:
Or you can place appropriately named icons in your global icon theme.
You mean I have to slice tango_icons.svg to a lot of icons and put them somewhere inside /usr/share/inkscape/icons/?
Not there, but in your system-wide icon theme. Those are typically in /usr/share/icons. That's what all other Gnome applications do. Support for theming is a feature that has benefits only in the long run once people start incorporating icons for Inkscape into their themes.
Hopefully it was truly dead code, and not code that just pushed things
into the system so that other code could pull it out later. Otherwise that was the "magic" that allowed things to start working. We really need that "magic".
I think it was dead code, because SPIcon didn't actually look up stock items by the IDs that were defined in the headers. Additionally the stock icon code didn't even touch icons.svg. I think this was a part of the unfinished effort that left us with the unused EditWidget class.
Regards, Krzysztof Kosiński