<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/101585/">http://git.reviewboard.kde.org/r/101585/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 6th, 2011, 1:31 p.m., <b>Aaron J. Seigo</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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 the dialog rather wide now. i wonder if it wouldn't be nicer to make Remove Project a button with a menu containing "From project list" and "From disk" as entries? this would make the window smaller and reduce some visual duplication ("Remove Project" x 2) while nicely clustering the options.</pre>
</blockquote>
<p>On July 6th, 2011, 7:07 p.m., <b>Giorgos Tsiapaliwkas</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">a KComboBox should do the work.Do you agree?
Do you want to remove all the relevant code for cancelRemoveButton?
Imo,i just wrote this code cuz it was a TODO,since there are the KMessageBoxes i thing that it is useless to have a qcheckbox.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">you don't need a KComboBox.. you can add a QMenu (or a KMenu, for that matter) to [QK]PushButton with QPushButton::setMenu(QMenu *menu).
and yes, i think all the checkbox related code can go.</pre>
<br />
<p>- Aaron J.</p>
<br />
<p>On July 6th, 2011, 1:16 p.m., Giorgos Tsiapaliwkas wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Giorgos Tsiapaliwkas.</div>
<p style="color: grey;"><i>Updated July 6, 2011, 1:16 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">hello,
the following patch is a summary of some other patches that i had to my local repository.Because it is too difficult to make different request due to share code i made 1 review request.(Sorry for the inconvinience.)
This patch shares code with these two.
git.reviewboard.kde.org/r/101535/
git.reviewboard.kde.org/r/101584/
The patch adds a Qcheckbox which enables the user to protect one project from the removal.
Also the delete buttons were able to be used even if a project wasn't selected at all,with this patch not anymore.
The code lacks only in the storage possibility.Plasmate is not able to "remember" which projects are selected or not,i will fix this later.
P.S.:i created the checkbox with some intelligence in order to work only when it is supposed to work.For example,only when the user selects a project.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">compiles fines,tested by me.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>mainwindow.cpp <span style="color: grey">(2ca2b64)</span></li>
<li>projectmanager/projectmanager.h <span style="color: grey">(d24ef19)</span></li>
<li>projectmanager/projectmanager.cpp <span style="color: grey">(42a28f3)</span></li>
<li>startpage.h <span style="color: grey">(6d45e02)</span></li>
<li>startpage.cpp <span style="color: grey">(3d0c0d9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101585/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>