<div dir="ltr">On 13 January 2014 21:13, Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org" target="_blank">manuq@laptop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2014/1/11 Daniel Narvaez <<a href="mailto:dwnarvaez@gmail.com">dwnarvaez@gmail.com</a>>:<br>
<div class="im">> Ok, we looked more into this and it's an horrible, horrible mess. Several<br>
> activities are using Gtk.IconSize, often just passing it to Gtk.Image. Here<br>
> is the best possible plan I can think of:<br>
<br>
</div>I agree with the whole plan.<br>
<div class="im"><br>
><br>
> - Change sugar and sugar-toolkit-gtk3 to never use Gtk.IconSize explicitly.<br>
> Just use pixels, the sizes stuff is half deprecated and of very little use.<br>
> - Set the proper pixel_size on the toolbutton image.<br>
<br>
</div>I'll fix sugar and sugar-toolkit-gtk3 basing on Ignacio work<br>
<div class="im"><br>
> Hacks to help backward compatibility<br>
><br>
> - Register sugar specific sizes (LARGE_TOOLBAR, MENU) with the deprecated<br>
> Gtk.icon_size_register function.<br>
> - Overwrite Gtk.IconSize.* with the sugar specific sizes. This will only<br>
> work at python level, but should help a lot with activities that uses them.<br>
<br>
</div>I came up with this:<br>
<a href="https://github.com/manuq/sugar-toolkit-gtk3/pull/4/files" target="_blank">https://github.com/manuq/sugar-toolkit-gtk3/pull/4/files</a><br></blockquote><div><br></div><div>Thanks! I commented there. Not sure if it was intentional but this was a pull request to your own repo, not to upstream. <br>
</div></div></div></div>