<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://reviewboard.kde.org/r/4513/">http://reviewboard.kde.org/r/4513/</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 3rd, 2010, 8:14 p.m., <b>Todd</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre>This probaly also needs to be fixed in kdelibs renamedialog, which also excludes the extension from selections..</pre>
</blockquote>
</blockquote>
<pre>I will take a look at that, too, then. Thank you!</pre>
<br />
<p>- Christian</p>
<br />
<p>On July 3rd, 2010, 4:26 a.m., Christian Muehlhaeuser wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 and Fredrik Höglund.</div>
<div>By Christian Muehlhaeuser.</div>
<p style="color: grey;"><i>Updated 2010-07-03 04:26:27</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;">As discussed with FredrikH:
When renaming an item, KFileItemDelegate excludes the extension from the initial text selection.
While this makes perfect sense for files, directories don't typically have an extension, which indicates a filetype. They're directories after all.
This patch keeps the existing behavior for files, but always selects the entire text for directories.
Note, that I'm afraid I couldn't test the patch compiling nicely on 4.5 yet.
Since it's "just a two-liner (tm)", I'm confident it works fine as attached here and I'd hope someone on here could confirm it compiling / working for me, please?
Please don't commit the patch just yet. For consistency's sake I'm also fixing the behavior inside Dolphin's own RenameDialog right now. I'm going to commit
both patches at the same time, then.
Thanks in advance,
Chris</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>trunk/KDE/kdelibs/kio/kio/kfileitemdelegate.cpp <span style="color: grey">(1145462)</span></li>
</ul>
<p><a href="http://reviewboard.kde.org/r/4513/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>