[Sugar-devel] Gtk 3.10 icon size regression

Daniel Narvaez dwnarvaez at gmail.com
Mon Jan 13 10:36:11 EST 2014


On 13 January 2014 16:19, Gonzalo Odiard <godiard at sugarlabs.org> wrote:

>
>> To make sure we are on the same page... I think the icon_size parameter
>> should be supported (by converting internally to the appropriate pixel
>> size) but deprecated. And thus patches that get rid of icon_size usage in
>> core or activities are welcome.
>>
>
> My point was about the naming of the parameter only, but I assume, we will
> be setting the size in pixels,
> and the constants defined in the toolkit will calculate the number of
> pixels needed according to screen resolution,
> then, do not have too much sense discuss about it.
>

If we was designing the API today I would just call the parameter size I
think, but the documentation would say it is pixel.

It's a bit of a complicated discussion because we have not agreed on a
clear plan on resolution independence going forward. Though, as I see it,
there are two things that will affect the physicall pixel size of an icon

1 The "theme" which is used. i.e. the value assigned to
style.STANDARD_ICON_SIZE for example.
2 The ratio between the logical pixels and the physical pixels.

For similar devices, say standard laptop screens, we don't even need 1. So
I think it has to be possible to pass logical pixels to the API, not
everything goes necessary through the theme.

(I'm mostly bringing this up for the sake of starting a discussion about
proper support for resolution independence)


> What we need is a way to publish these changes and what should be done,
> when is decided.
> (like with GCOnf to GSettings and other changes)
>

Yes, maybe we need a page on sugar-docs where, for each major release, we
list deprecated API's and suggest how to deal with it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20140113/a35ee78c/attachment.html>


More information about the Sugar-devel mailing list