<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="https://git.reviewboard.kde.org/r/117949/">https://git.reviewboard.kde.org/r/117949/</a>
     </td>
    </tr>
   </table>
   <br />



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ship It!</pre>
 <br />









<p>- Boudewijn Rempt</p>


<br />
<p>On May 2nd, 2014, 3:49 p.m. UTC, Friedrich W. H. Kossebau wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Calligra and Cyrille Berger Skott.</div>
<div>By Friedrich W. H. Kossebau.</div>


<p style="color: grey;"><i>Updated May 2, 2014, 3:49 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
calligra
</div>


<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 current filter dialog has a hiddable list of all filter (configurations). If the user opens the view, by moving the splitter on the left into the middle of the dialog, and then in that list selects another filter (configuration), that filter (configuration) is activated and the UI adapts to the configuration.
But it does not adapt the title of the dialog.

Attached patch fixes that.

The patch also prefixes the dialog title with some "Filter: ", to make it more obvious what this dialog is about.

KisFilterSelectorWidget::currentFilterName() is removed completely and the only current caller changed to query that property itself. That method relied on callers only calling it if a filter was selected. So I kept that and also made that caller not check if there is a filter or not. Not sure if that should not be made more safe.</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>krita/ui/dialogs/kis_dlg_adjustment_layer.cc <span style="color: grey">(13f4e70)</span></li>

 <li>krita/ui/dialogs/kis_dlg_filter.h <span style="color: grey">(cc103ef)</span></li>

 <li>krita/ui/dialogs/kis_dlg_filter.cpp <span style="color: grey">(844ab1d)</span></li>

 <li>krita/ui/widgets/kis_filter_selector_widget.h <span style="color: grey">(0b99032)</span></li>

 <li>krita/ui/widgets/kis_filter_selector_widget.cc <span style="color: grey">(bd08586)</span></li>

</ul>

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







  </td>
 </tr>
</table>








  </div>
 </body>
</html>