The issue with using the browse object, is have the same colors in the theme,<div>and the colors are applied to this particular class.</div><div>Probably is a good moment to think if should not be applied to the Notebook, or create a more general class like SugarNotebook.</div>
<div>I think these ports will be great to help us to found a lot of these issues.</div><div><br></div><div>Gonzalo</div><div><br><br><div class="gmail_quote">On Tue, May 15, 2012 at 11:09 AM, Manuel Kaufmann <span dir="ltr"><<a href="mailto:humitos@gmail.com" target="_blank">humitos@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<div class="im"><br>
On Mon, May 14, 2012 at 4:32 PM, Agustin Zubiaga <<a href="mailto:aguz@sugarlabs.org">aguz@sugarlabs.org</a>> wrote:<br>
> diff --git a/terminal.py b/terminal.py<br>
> index 160b35d..59b4940 100644<br>
<br>
</div><div class="im">> +from widgets import BrowserNotebook, TabLabel<br>
<br>
</div>Please, use one line per import here.<br>
<div class="im"><br>
> -        self._notebook = Notebook()<br>
> +        self._notebook = BrowserNotebook()<br>
> +        self._notebook.connect("tab-added", self.__open_tab_cb)<br>
<br>
</div>Just asking: Is it OK using the name BrowseNotebook for this class?<br>
<div class="im"><br>
> +        tablabel = TabLabel()<br>
> +        tablabel.connect('tab-close', self.__close_tab_cb)<br>
> +        box.label = tablabel<br>
> +<br>
> +        index = self._notebook.append_page(box, tablabel)<br>
> +        tablabel.show_all()<br>
<br>
</div>I would use: tab_label as variable name :)<br>
<div class="im"><br>
> diff --git a/widgets.py b/widgets.py<br>
> new file mode 100644<br>
> index 0000000..93d7dc0<br>
</div>> +        self.set_name('browse-tab-add')<br>
<br>
This line comes from Browse that is why this has "browse" inside the<br>
string. Is it OK?<br>
<br>
> +        button.set_name('browse-tab-close')<br>
<br>
The same happens here.<br>
<br>
<br>
THANKS a lot for your patch!<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Kaufmann Manuel<br>
Blog: <a href="http://humitos.wordpress.com/" target="_blank">http://humitos.wordpress.com/</a><br>
Porfolio: <a href="http://fotos.mkaufmann.com.ar/" target="_blank">http://fotos.mkaufmann.com.ar/</a><br>
PyAr: <a href="http://www.python.com.ar/" target="_blank">http://www.python.com.ar/</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></div>