On Fri, Mar 25, 2011 at 7:43 PM, Dan Winship <[email protected]> wrote: > On 03/25/2011 02:31 PM, Giovanni Campagna wrote: >> 0001: Isn't this calling _reposition for every frame? Every call to >> _reposition queues a relayout, which in turns result in _allocate, and >> then to queue a _reposition. > > No, because if nothing changed, then _reposition will set actor.x and > actor.y to the same values they already had, and ClutterActor will just > optimize that out and not queue another relayout.
Ok. In that case it is good to commit. >> 0003: The fix is consistent with current background-image support, and >> good as a bandaid, until background-position / background-size / >> background-repeat are supported. But I'm not sure it does fix any of the >> sizing bug, given that the font size does not change > > Ah... I hadn't noticed that it was correct under the default font > scaling. But anyway, the font size does change if you do Accessibility > -> Large Text. At Large it's a little bit off. At Larger (which you have > to go to the a11y control panel for), it's quite broken. Ah, I didn't consider Large Text. Then it's better to fix it. Giovanni _______________________________________________ gnome-shell-list mailing list [email protected] http://mail.gnome.org/mailman/listinfo/gnome-shell-list
