<div dir="ltr">On 22 November 2013 12:49, 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">
Hi,<br>
<br>
sorry for not joining the discussion yet.  I'm a bit overloaded with<br>
non-Sugar stuff.<br></blockquote><div><br></div><div>Ufff, we need you full time! :P<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I remember Agustín Zubiaga also mentioned Box(orientation) instead of<br>
HBox, VBox at the time of the GTK3 port, and then I came to read about<br>
Grid. When I checked, the GTK docs were recommending it, but most of<br>
their code was not using it, and the API looked weird to me in Python,<br>
less readable.  That is for a replacement of simple, flat boxes.  But<br>
if Grid allows to remove the boxes nesting in Sugar widgets like<br>
Palette, then we definetely should use it.<br></blockquote><div><br></div><div>Perhaps the reason their code was not using it is that it will take a while to get converted. The API is pretty nice for simple boxes really, it's more that the reference documentation is doing a bad job at showing it.<br>
<br></div><div>box = Gtk.Grid(orientation=Gtk.Orientation.HORIZONTAL, spacing=20)<br><br></div><div>widget1 = MyWidget(margin=5)<br></div><div>box.add(widget1)<br></div><div>widget1.show()<br><br></div><div>widget2 = MyWidget(margin_top=10, margin_left=5)<br>
</div><div>box.add(widget2)<br></div><div>widget2.show()<br></div><div><br></div><div>box.show() <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

For the widget spacings and dimentions, keep in mind that part is done<br>
in the Python code, and part in the artwork CSS. And some things can<br>
be done in the two sides.<br>
<br>
Example:<br>
<br>
grid margin (Python):<br>
<a href="https://developer.gnome.org/gtk3/stable/GtkWidget.html#GtkWidget--margin" target="_blank">https://developer.gnome.org/gtk3/stable/GtkWidget.html#GtkWidget--margin</a><br>
margin (CSS) <a href="https://developer.gnome.org/gtk3/stable/GtkStyleContext.html#GTK-STYLE-PROPERTY-MARGIN:CAPS" target="_blank">https://developer.gnome.org/gtk3/stable/GtkStyleContext.html#GTK-STYLE-PROPERTY-MARGIN:CAPS</a><br>

separator height (CSS):<br>
<a href="https://developer.gnome.org/gtk3/stable/GtkWidget.html#GtkWidget--s-separator-height" target="_blank">https://developer.gnome.org/gtk3/stable/GtkWidget.html#GtkWidget--s-separator-height</a><br>
<br>
What I've been doing in styling is: use CSS as much as possible.  If<br>
there is a CSS way and also a programmatic way, use CSS.<br></blockquote><div><br></div><div>I agree. I suppose it's possible to easily use CSS for non-toolkit stuff too? (activities and shell).<br></div><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
To check graphics regressions is good to do visual comparisons for example:<br>
<a href="http://bugs.sugarlabs.org/attachment/ticket/4295/palette-border-wrong.gif" target="_blank">http://bugs.sugarlabs.org/attachment/ticket/4295/palette-border-wrong.gif</a><br></blockquote><div><br></div><div>Good idea.<br>
</div><div><br></div><div>Also we really need to start writing regression tests for sugar.graphics. Just render offscreen and compare, it should be easy to set that up. <br></div></div></div></div>