Re: Review Request: Plasmate:qcheckbox added and the removal buttons became more intellect.

2011-07-07 Thread Aaron J. Seigo
On July 6, 2011, 1:31 p.m., Aaron J. Seigo wrote: what is the use case for the checkbox? it is nicely implemented and all, but i'm not sure what the real value of it is, versus the cost of having it in the user interface and making it more complex? also, the set of buttons makes

Re: Review Request: Plasmate:qcheckbox added and the removal buttons became more intellect.

2011-07-06 Thread Giorgos Tsiapaliwkas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101585/ --- (Updated July 6, 2011, 1:16 p.m.) Review request for Plasma. Changes

Re: Review Request: Plasmate:qcheckbox added and the removal buttons became more intellect.

2011-07-06 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101585/#review4463 --- what is the use case for the checkbox? it is nicely implemented

Re: Review Request: Plasmate:qcheckbox added and the removal buttons became more intellect.

2011-07-06 Thread Giorgos Tsiapaliwkas
On July 6, 2011, 1:31 p.m., Aaron J. Seigo wrote: what is the use case for the checkbox? it is nicely implemented and all, but i'm not sure what the real value of it is, versus the cost of having it in the user interface and making it more complex? also, the set of buttons makes

Re: Review Request: Plasmate:qcheckbox added and the removal buttons became more intellect.

2011-06-16 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101585/#review3928 --- some remarks in here, some also apply to your other request.

Review Request: Plasmate:qcheckbox added and the removal buttons became more intellect.

2011-06-11 Thread Giorgos Tsiapaliwkas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101585/ --- Review request for Plasma. Summary --- hello, the following patch