<div dir="ltr">On 13 January 2014 15:37, Gonzalo Odiard <span dir="ltr"><<a href="mailto:godiard@sugarlabs.org" target="_blank">godiard@sugarlabs.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Sadly, have more sense set icon_size than pixel_size, right?<div>(More in the context of multiple pixels resolutions, like we have with the xo and the desktop)<br>
</div></div></blockquote><div><br></div>Well, we should be using the new layout scaling stuff in gtk to deal with different resolutions.<br><br><a href="http://blogs.gnome.org/alexl/2013/06/28/hidpi-support-in-gnome/">http://blogs.gnome.org/alexl/2013/06/28/hidpi-support-in-gnome/</a><br>
<br>If on the top of that we need to set different "pixel" sizes for icons, it seems like a constant will probably enough. The IconSize stuff was designed to support runtime switching because it was user configurable, and it's now all being deprecated because they don't want the user configurability anymore. In any case gtk is moving away from it, so we will need our own constant or more complex thing.<br>
</div><div class="gmail_quote"> <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div></div><div>I think is better fix Icon to get the parameter and set the pixel_size,</div>

<div>and not do all the modifications Ignacio found.</div></div></div></blockquote><div><br></div><div>Why? I think it's pretty clear that Gtk.IconSize will go away in gtk4. It seems a good idea to get rid of it at least in core (and suggest to do the same in activities). It will be less work when it goes away completely.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div><div>In fact, Icon is sugar toolkit code, and do other tricks,</div>
<div>then we don't need deprecate the parameter.</div><span class=""></span></div></div></blockquote><div><br></div><div>But Gtk.IconSize is not sugar toolkit API. "Use Gtk.IconSize with Icon but not with gtk.Image" seems like a very very confusing thing to suggest to activity authors.<br>
</div></div></div></div>