Re: Review Request 108562: Let calligraphy tool use foreground color and cross cursor

2013-01-22 Thread Boudewijn Rempt
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108562/#review26055 --- Ship it! Maybe the calligraphy tool was supposed to have an ou

Re: Review Request 108563: Finish removing changetracker code from KoTextWriter

2013-01-22 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108563/#review26047 --- Ship it! Ship It! - C. Boemann On Jan. 23, 2013, 5:41 a.m.,

Re: Review Request 108563: Finish removing changetracker code from KoTextWriter

2013-01-22 Thread Luke De Mouy
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108563/ --- (Updated Jan. 23, 2013, 5:41 a.m.) Review request for Calligra. Changes

Re: Review Request 108557: Do not pass KoDocument as argument to openDocument

2013-01-22 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108557/#review26045 --- Ship it! Ship It! - C. Boemann On Jan. 22, 2013, 7:32 p.m.,

Re: Review Request 108563: Finish removing changetracker code from KoTextWriter

2013-01-22 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108563/#review26044 --- libs/kotext/opendocument/KoTextWriter_p.h

Review Request 108563: Finish removing changetracker code from KoTextWriter

2013-01-22 Thread Luke De Mouy
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108563/ --- Review request for Calligra. Description --- Finish removing change t

Review Request 108562: Let calligraphy tool use foreground color and cross cursor

2013-01-22 Thread Sven Langkamp
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108562/ --- Review request for Calligra. Description --- Patch lets the calligrap

Review Request 108557: Do not pass KoDocument as argument to openDocument

2013-01-22 Thread Gopalakrishna Bhat
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108557/ --- Review request for Calligra. Description --- Remove the KoDocument as

Re: Review Request 108481: Temporarily swith to path tool to change shape parameters without double click to switch tool

2013-01-22 Thread Thorsten Zachmann
> On Jan. 20, 2013, 8:49 a.m., C. Boemann wrote: > > I have some reservations against this. > > > > In general I think this is the way we should go but I think we should come > > up with a general solution instead of patchinga single use case > > > > I've been trying to do that grand solution

Re: Review Request 108459: Refactor in KoText

2013-01-22 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108459/ --- (Updated Jan. 22, 2013, 2:17 p.m.) Review request for Calligra and Thorste