<br><br><div class="gmail_quote">On Tue, Jun 26, 2012 at 9:20 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Thanks for the patch.  I have review comments for one change:<br>
<br>
On Tue, Jun 26, 2012 at 10:53:57PM -0300, Agustin Zubiaga wrote:<br>
> [...]<br>
<div class="im">> diff --git a/readtopbar.py b/readtopbar.py<br>
> index d0da3d0..b6fe196 100644<br>
> --- a/readtopbar.py<br>
> +++ b/readtopbar.py<br>
> @@ -33,15 +33,16 @@ _UP_DEVICE_IFACE = 'org.freedesktop.UPower.Device'<br>
>  _UP_TYPE_BATTERY = 2<br>
><br>
><br>
> -class _TopBar(Gtk.HBox):<br>
> +class _TopBar(Gtk.Box):<br>
>      __gproperties__ = {<br>
>          'completion-level': (float, None, None, 0.0, 100.0, 0.0,<br>
>                               GObject.ParamFlags.WRITABLE),<br>
>      }<br>
><br>
>      def __init__(self):<br>
> -        Gtk.HBox.__init__(self)<br>
> +        Gtk.Box.__init__(self)<br>
><br>
> +        self.set_orientation(Gtk.Orientation.VERTICAL)<br>
>          self.set_border_width(int(style.DEFAULT_SPACING / 2.0))<br>
>          self.set_spacing(style.DEFAULT_SPACING * 4)<br>
><br>
<br>
</div>In this change you move from an Horizontal Box to a Box, but set the<br>
orientation to vertical.  I could not tell if this was desired.  If<br>
this is a desired change, perhaps it should be mentioned in the patch<br>
comment.<br>
<br>
The other changes looked fine.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Same comment as James, other than that is +1.</div><div>.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888">
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</div></div></blockquote></div><br>