Re: Review Request: User defined variables (2)

2012-01-07 Thread C. Boemann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/#review9637 --- Was this not already merged (but disabled). if so then please cl

Re: Review Request: User defined variables (2)

2011-12-05 Thread Thorsten Zachmann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/#review8721 --- Ship it! Looks good. I say ship it but it contains some new str

Re: Review Request: User defined variables (2)

2011-12-05 Thread Sebastian Sauer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/ --- (Updated Dec. 5, 2011, 9:50 a.m.) Review request for Calligra and Thorsten

Re: Review Request: User defined variables (2)

2011-12-03 Thread Thorsten Zachmann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/#review8692 --- Hello Sebastian, the new patch seems to be the same as the old

Re: Review Request: User defined variables (2)

2011-12-01 Thread Sebastian Sauer
> On Nov. 28, 2011, 6:53 a.m., Thorsten Zachmann wrote: > > plugins/variables/UserVariable.h, lines 65-67 > > > > > > Is it possible to move the option widget into its own class. That makes > > it easier for applica

Re: Review Request: User defined variables (2)

2011-12-01 Thread Sebastian Sauer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/ --- (Updated Dec. 1, 2011, 7:13 p.m.) Review request for Calligra and Thorsten

Re: Review Request: User defined variables (2)

2011-12-01 Thread Sebastian Sauer
> On Nov. 28, 2011, 6:53 a.m., Thorsten Zachmann wrote: > > libs/kotext/KoVariableManager.cpp, line 137 > > > > > > This should not use namedItemNS as this will iterate over all elements > > which can be very long

Re: Review Request: User defined variables (2)

2011-11-28 Thread Sebastian Sauer
> On Nov. 28, 2011, 6:53 a.m., Thorsten Zachmann wrote: > > Looks quite good. I have not yet tested it. It would be good if Mek could > > have a look at the calligra tables changes. I removed the Calligra Tables integration again. There is no useful way to add/remove/edit/display them atm anyw

Re: Review Request: User defined variables (2)

2011-11-27 Thread Thorsten Zachmann
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/#review8555 --- Looks quite good. I have not yet tested it. It would be good if

Review Request: User defined variables (2)

2011-11-27 Thread Sebastian Sauer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103276/ --- Review request for Calligra and Thorsten Zachmann. Description --- Fo