Thanks Tav, that's great news. Completely agreed that we should unify a lot of our icon handling code. Krzysztof and I discussed this briefly at the Hackfest. I think we should seriously consider moving towards a more standard way of installing our icon theme.
Our current method of pulling multiple icons in from a single SVG document is perhaps a more "Inkscape-esque" way of doing things than reading from individual SVG/bitmap icon files. However, it's not very friendly for GNOME theme designers and requires quite a lot of custom code rather than being handled simply through a GtkIconTheme instance.
AV
On 4 May 2016 at 10:35, Tavmjong Bah <tavmjong@...8...> wrote:
On Tue, 2016-05-03 at 00:43 -0700, Krzysztof Kosiński wrote:
I added some more information.
I think the icon clipping bug may be related to the use of SPIcon. If we can replace that with GtkImage, it might go away.
I fixed this in r14870. The icon clipping is due to wrapping the icon in a widget where the preferred width/height of the widget was calculated by adding the preferred width/height of the icon to the border width. By also adding the padding width the icons are no longer clipped.
I am not sure why we need to be concerned with border/padding widths. In a style sheet based framework, border/padding/margin should be taken care of automatically.
One thing I observed in tracking this problem down is that we have many ways to turn icons into buttons. We should try to unify these as we move to GTK3.
Tav
Find and fix application performance issues faster with Applications Manager Applications Manager provides deep performance insights into multiple tiers of your business applications. It resolves application problems quickly and reduces your MTTR. Get your free trial! https://ad.doubleclick.net/ddm/clk/302982198;130105516;z _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel