<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://svn.reviewboard.kde.org/r/5443/">http://svn.reviewboard.kde.org/r/5443/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I liked the idea of telling the user if he's overwriting an older file (usually fine) or a newer file (usually an error). Indeed it made the dialog very wide, so it could be done better. Maybe simply adding a second line "Warning, the destination file is newer!" when this is the case, and not saying anything when it's older or same age?</pre>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://svn.reviewboard.kde.org/r/5443/diff/5/?file=38952#file38952line218" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/KDE/kdelibs/kio/kio/renamedialog.cpp</a>
<span style="font-weight: normal;">
(Diff revision 5)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">RenameDialog::RenameDialog(QWidget *parent, const QString & _caption,</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">214</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">connect</span><span class="p">(</span><span class="n">srcJob</span><span class="p">,</span> <span class="n">SIGNAL</span><span class="p">(</span><span class="n">gotPreview</span><span class="p">(</span><span class="k">const</span> <span class="n">KFileItem</span><span class="o">&</span><span class="p">,</span> <span class="k">const</span> <span class="n">QPixmap</span><span class="o">&</span><span class="p">)),</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">218</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">connect</span><span class="p">(</span><span class="n">srcJob</span><span class="p">,</span> <span class="n">SIGNAL</span><span class="p">(</span><span class="n">gotPreview</span><span class="p">(</span> <span class="k">const</span> <span class="n">KFileItem</span><span class="o">&</span><span class="p">,</span> <span class="k">const</span> <span class="n">QPixmap</span><span class="o">&</span><span class="p">)</span> <span class="p">),</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Adding spaces in SIGNAL() only makes this slower since Qt will have to remove them.</pre>
</div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://svn.reviewboard.kde.org/r/5443/diff/5/?file=38952#file38952line229" style="color: black; font-weight: bold; text-decoration: underline;">/trunk/KDE/kdelibs/kio/kio/renamedialog.cpp</a>
<span style="font-weight: normal;">
(Diff revision 5)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">RenameDialog::RenameDialog(QWidget *parent, const QString & _caption,</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">225</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">QScrollArea</span><span class="o">*</span> <span class="n">srcWidget</span> <span class="o">=</span> <span class="n">createContainerLayout</span><span class="p">(</span><span class="n">parent</span><span class="p">,</span> <span class="n">srcItem</span><span class="p">,</span> <span class="n">d</span><span class="o">-></span><span class="n">m_srcPreview</span><span class="p">);</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">228</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">QScrollArea</span><span class="o">*</span> <span class="n">srcWidget</span> <span class="o">=</span> <span class="n">createContainerLayout</span><span class="p">(</span> <span class="n">parent</span><span class="p">,</span> <span class="n">srcItem</span><span class="p">,</span> <span class="n">d</span><span class="o">-></span><span class="n">m_srcPreview</span> <span class="p">);</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Why is the only change to this line, the addition of spaces inside (), which is actually against the kdelibs coding style? Yes the file was probably not consistent before, but the right direction to go is to remove these spaces in the code you touch, not to add them ;)
http://techbase.kde.org/Policies/Kdelibs_Coding_Style</pre>
</div>
<br />
<p>- David</p>
<br />
<p>On October 5th, 2010, 5:05 p.m., Steven Sroka wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 kdelibs.</div>
<div>By Steven Sroka.</div>
<p style="color: grey;"><i>Updated 2010-10-05 17:05:06</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;">This is my first submission to KDE. If I am missing something, don't hesitate to tell me.
This is a slight GUI change to the rename/overwrite dialog window, just to make it more user friendly.</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;">Correctly Compiled.
"Eye ball'ed" Code.
Ran code with all possible '_mode' as per http://api.kde.org/4.5-api/kdelibs-apidocs/kio/html/namespaceKIO.html#bac5df6792cd3426582dbfd1af706bff
Ran many possible combinations (most if not all) -> move folder to folder, file to file, file to folder, folder to file, and paid attention to creation date.
(I actually found a bug with the preview picture that is shown in a certain scenario - I will create a bug notice for it on bugs.kde.org soon)</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=238942">238942</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/KDE/kdelibs/kio/kio/renamedialog.h <span style="color: grey">(1181964)</span></li>
<li>/trunk/KDE/kdelibs/kio/kio/renamedialog.cpp <span style="color: grey">(1181964)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/5443/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://svn.reviewboard.kde.org/r/5443/s/524/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/10/03/Trunk_-_Folder_to_Folder_400x100.png" style="border: 1px black solid;" alt="KDE Trunk - Folder to Folder" /></a>
<a href="http://svn.reviewboard.kde.org/r/5443/s/525/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/10/03/New_-_Folder_to_Folder_400x100.png" style="border: 1px black solid;" alt="*New - Folder to Folder" /></a>
<a href="http://svn.reviewboard.kde.org/r/5443/s/526/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/10/03/Trunk_-_Folder_to_File_400x100.png" style="border: 1px black solid;" alt="KDE Trunk - Bug" /></a>
<a href="http://svn.reviewboard.kde.org/r/5443/s/527/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/10/03/New_-_File_to_File_400x100.png" style="border: 1px black solid;" alt="*New - File to File" /></a>
<a href="http://svn.reviewboard.kde.org/r/5443/s/528/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/10/03/KDE4.5__Preview_400x100.png" style="border: 1px black solid;" alt="KDE 4.5 - File to File" /></a>
<a href="http://svn.reviewboard.kde.org/r/5443/s/529/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/10/04/Rev5_Image_400x100.png" style="border: 1px black solid;" alt="Revision 5 Screenshot" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>