Re: [Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread James Cameron
On Wed, Jun 27, 2012 at 12:46:00AM -0300, Agustin Zubiaga Sanchez wrote: > Ok, In this message I send my patch again but without this line. Reviewed-by: James Cameron (Also tested that it applies cleanly to master). -- James Cameron http://quozl.linux.org.au/ __

Re: [Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread Agustin Zubiaga Sanchez
Ok, In this message I send my patch again but without this line. Regards, aguz 2012/6/27 James Cameron > On Wed, Jun 27, 2012 at 12:19:55AM -0300, Agustin Zubiaga Sanchez wrote: > > sorry, I didn't want to do it. > > Can you remove it? > > No, not me. > > This brings up an interesting process q

Re: [Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread James Cameron
On Wed, Jun 27, 2012 at 12:19:55AM -0300, Agustin Zubiaga Sanchez wrote: > sorry, I didn't want to do it. > Can you remove it? No, not me. This brings up an interesting process question. If I was a committer, I would modify and apply the patch. You would then have to merge the conflict. Howeve

Re: [Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread Agustin Zubiaga Sanchez
Hi, James sorry, I didn't want to do it. Can you remove it? 2012/6/27 Rafael Ortiz > > > On Tue, Jun 26, 2012 at 9:20 PM, James Cameron wrote: > >> Thanks for the patch. I have review comments for one change: >> >> On Tue, Jun 26, 2012 at 10:53:57PM -0300, Agustin Zubiaga wrote: >> > [...] >>

Re: [Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread Rafael Ortiz
On Tue, Jun 26, 2012 at 9:20 PM, James Cameron wrote: > Thanks for the patch. I have review comments for one change: > > On Tue, Jun 26, 2012 at 10:53:57PM -0300, Agustin Zubiaga wrote: > > [...] > > diff --git a/readtopbar.py b/readtopbar.py > > index d0da3d0..b6fe196 100644 > > --- a/readtopba

Re: [Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread James Cameron
Thanks for the patch. I have review comments for one change: On Tue, Jun 26, 2012 at 10:53:57PM -0300, Agustin Zubiaga wrote: > [...] > diff --git a/readtopbar.py b/readtopbar.py > index d0da3d0..b6fe196 100644 > --- a/readtopbar.py > +++ b/readtopbar.py > @@ -33,15 +33,16 @@ _UP_DEVICE_IFACE = '

[Sugar-devel] [PATCH Read] Use Box, ButtonBox, Scrollbar instead of his deprecated form

2012-06-26 Thread Agustin Zubiaga
Signed-off-by: Agustin Zubiaga --- epubview/epubview.py |9 + readactivity.py | 11 +++ readdialog.py| 11 +++ readsidebar.py |6 -- readtoolbar.py |3 ++- readtopbar.py|5 +++-- sugarmenuitem.py |5 +++-- 7