<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/106289/">http://git.reviewboard.kde.org/r/106289/</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;">Thanks for your explanations!

First of all, let me say that I greatly appreciate all the awesome work you're doing in Konqueror and kdelibs. I know that you made many great contributins to areas that many others have little interest in.

I see now why you propose to add these signals, but I ask you to also try to understand my point of view. One of my main goals is to keep Dolphin's code readable and maintainable. If we follow your suggestion, KFileItemModel would have the signals

itemsAdded(const KFileItemList&)

itemsDeleted(const KFileItemList&)

itemsInserted(const KItemRangeList&)

itemsRemoved(const KItemRangeList&)

which have completely different semantics. This would be quite confusing, not only at first sight, and seriously harm the readability of the code IMHO.

But I think that you can achieve what you want quite easily without these signals. You could create a dir lister inside the dir filter plugin that watches the directory. This would give you access to all files in the directory without the need to add those signals. You would just have to make sure that this dir lister has the correct "show hidden files" setting, but this should be doable because DolphinView has a signal hiddenFilesShownChanged(bool).

If this solution is acceptable for you, I'm happy to add the mime filter functionality to KFileItemModel.

Just for the record, I also discussed this with Peter last night, just to make sure that I don't tell you complete nonsense here. He agrees that adding those signals to KFileItemModel and DolphinView would be a very bad idea.</pre>

 <br />

<p>- Frank</p>

<br />

<p>On September 3rd, 2012, 9:03 p.m., Dawit Alemayehu 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 Dolphin and KDE Base Apps.</div>

<div>By Dawit Alemayehu.</div>

<p style="color: grey;"><i>Updated Sept. 3, 2012, 9:03 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;">The attached patch provides an implementation of KParts' ListingFilterExtension for Dolphin. The extension allows the Dolphin KPart to provide directory/file filtering services without requiring direct linking against Dolphin itself.

The review for the new KPart listing filter extension (ListingFilterExtension) can be found at https://git.reviewboard.kde.org/r/106288/</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>dolphin/src/views/dolphinview.h <span style="color: grey">(10f63c5)</span></li>

 <li>dolphin/src/views/dolphinview.cpp <span style="color: grey">(8050415)</span></li>

 <li>dolphin/src/kitemviews/private/kfileitemmodelfilter.cpp <span style="color: grey">(816d356)</span></li>

 <li>dolphin/src/kitemviews/private/kfileitemmodelfilter.h <span style="color: grey">(9bdf1fd)</span></li>

 <li>dolphin/src/dolphinpart.h <span style="color: grey">(e5693b3)</span></li>

 <li>dolphin/src/dolphinpart.cpp <span style="color: grey">(fff7dc0)</span></li>

 <li>dolphin/src/kitemviews/kfileitemmodel.h <span style="color: grey">(d9bebdf)</span></li>

 <li>dolphin/src/kitemviews/kfileitemmodel.cpp <span style="color: grey">(6936af4)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/106289/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>