<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/106288/">http://git.reviewboard.kde.org/r/106288/</a>
</td>
</tr>
</table>
<br />
<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 and David Faure.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated Sept. 4, 2012, 6:09 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">- Split the filtering and notification into separate extension. Now there is a ListingFilterExtension and a ListingNotificationExtension. Though related they should not have been combined.
- Renamed the file from listingfilterextension.* to listingextension.*</pre>
</td>
</tr>
</table>
<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 adds a new extension, ListingFilterExtension, to KParts. The main motivation for this patch is the constant breakage of the Konqueror directory filtering plugin once Konqueror was switched to use Dolphin's part for filemanagement. Because the dolphin part does not expose the same features as the implementation in the old Konqueror file management module, I had to resort to ugly hacks to make this plugin functional. Unfortunately, that hack no longer works properly because of changes/cleanup done in Dolphin's code base. Hence, this patch is an attempt to fix this issue once and for all in a manner that is clean, consistent and useful for any part that wants to provide filtering functionality. As an additional benefit this patch will also makes it very easy to add Dolphin's CTRL+I filtering feature to Konqueror.
The implementation of this extension in Dolphin can be reviewed at https://git.reviewboard.kde.org/r/106289/</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>includes/CMakeLists.txt <span style="color: grey">(8a956ee)</span></li>
<li>includes/KParts/ListingExtension <span style="color: grey">(PRE-CREATION)</span></li>
<li>kparts/CMakeLists.txt <span style="color: grey">(96fa31f)</span></li>
<li>kparts/listingextension.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kparts/listingextension.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106288/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>