
bulia byak wrote:
Jon, now the left toolbar buttons are almost the right size. A bit smaller, but again, if it's a standard size, let it be. (But if there's another standard size a bit larger than this, I'd like to use it instead.)
Yes, that's the standard "large toolbar" size. Now is probably the point at which you tweak that size for Inkscape in your gtkrc, or switch to a theme that has sizes you like. Or make your own theme. :-)
The icons on them, on the other hand, are larger than before. Can you please reduce them because: 1) they were pixel-optimized for the previous size and 2) they now look very cramped.
Hmmm... Don't think that's the way to go here. Since they're svg, we really shouldn't count on being optimized for 20 pixel instead of 24. Just switch to the "Large Print" theme and see what uses may like. Our UI should "just work" for both large print and bulia sizes.
The way things are working now is that I just ask GTK+ what size it has for icons on "large toolbars". I render a pixbuf at that size and return it. That's all. GTK+ takes over on the sizing of the buttons and all, according to it's rules and theme preferences.
Oh, and for figuring out, here is the list of logical icon sizes. Note that some of the sizes are the same as each other in different themes:
16 | 16 | 32 | GTK_ICON_SIZE_MENU 18 | 16 | 32 | GTK_ICON_SIZE_SMALL_TOOLBAR 24 | 24 | 48 | GTK_ICON_SIZE_LARGE_TOOLBAR 20 | 20 | 32 | GTK_ICON_SIZE_BUTTON 32 | 32 | 48 | GTK_ICON_SIZE_DND 48 | 48 | 64 | GTK_ICON_SIZE_DIALOG
The first column is the size by the generic query function. The second column is the size with the default theme and icons (notice that small toolbar changed), and the third is the size when in the 'Large Print' theme. What does this all show? Not to use hard sizes. Instead pick the enum the reflects the UI Widget's functional purpose. The if you don't like the given sizes, you can just override them in the theme or gtkrc.
Oh, and the huge eye and lock look really scary in the statusbar :) Please restore the sizes.
Yeah., I know. Eeeeeeeeeeeek!!!
That is what making me think that perhaps we should add a single enum value to get a 'decoration' or some such size. We won't be getting stock sizes for that, but we probably could figure something out. Does everyone think that's a good idea?