[Sugar-devel] Gtk 3.10 icon size regression

[Gonzalo Odiard]

+1


On Mon, Jan 13, 2014 at 12:36 PM, Daniel Narvaez <dwnarvaez at gmail.com>wrote:

> 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.
>



-- 
Gonzalo Odiard

SugarLabs - Learning Software for children
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20140113/7b948fdc/attachment.html>