<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/101013/">http://git.reviewboard.kde.org/r/101013/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 2nd, 2011, 10:49 p.m., <b>Christoph Feck</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;">Nice patch! See also bug 142900.
The duplicates of this bug mention that filtering does not work with folders. If that is fixable, it would be an added bonus :)</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;">Thanks for the bug report.
As for not filtering out directories, that behaviour seems to be hardcoded [1] in KDirLister (which KFileDialog uses via KFileWidget via KDirOperator). Such a change definitely seems like more than I'd be willing or able to undertake right now.
[1] https://projects.kde.org/projects/kde/kdelibs/repository/revisions/master/entry/kio/kio/kdirlister.cpp#L2305</pre>
<br />
<p>- Parker</p>
<br />
<p>On April 2nd, 2011, 11:47 p.m., Parker Coates 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 kdelibs.</div>
<div>By Parker Coates.</div>
<p style="color: grey;"><i>Updated April 2, 2011, 11:47 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;">This patch attempts to improve the usefulness and usability of KFileDialog's filter field when in Opening mode.
Firstly, if the filter text isn't: * the display name of one of the filters passed to the dialog or
* one or more space separated mimetype specifiers (containing a '/') or
* one or more space separated file globs (containing '*', '?' or "[.*]")
we convert the text to a glob by prepending and appending asterisks. This lets the user enter a piece of text (without having to know any glob patterns) and see only the files whose names contain that text, much the same as they would when filtering in Dolphin.
Secondly, the filtering updates on the fly as the filter text is typed. Previously, the filtering updated only when Return was pressed, which differs from the behaviour of most of KDE's other filter boxes. The old behaviour is especially confusing when one clicks the small clear button embedded in the combobox, because it clears the box, but the filtering is unchanged until the user goes to the keyboard to press enter.</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;">I've played around with it a fair bit and it seems to work fine. I've never really worked with this code before, so if I'm doing something silly please let me know.</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="http://bugs.kde.org/show_bug.cgi?id=142900">142900</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>kfile/kfilewidget.cpp <span style="color: grey">(9b8cdeb)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101013/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>