<div dir="ltr"><div><div>The patch is working good, cool!<br><br></div>Though as the filtering is case sensitive now, I'll have to rename all presets with all-lower-case to get rid of inconsistencies.<br></div><div>In the same time I'll also improve naming scheme to make filtering easier.<br>
</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/5/14  <span dir="ltr"><<a href="mailto:s_suelzer@lavabit.com" target="_blank">s_suelzer@lavabit.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Tue, May 14, 2013 at 06:33:52PM +0200, Sven Langkamp wrote:<br>
>    Overall pretty nice work.<br>
>    There are a few minor things I spotted:<br>
>    - that empty slot in PresetDockerDock can be removed completely including<br>
>    the connections<br>
>    - there were some coding styles issues with missing spaces between<br>
>    methods<br>
>    - KoPresetFiltering has a misleading name, at that level we don't have<br>
>    presets just resources<br>
>    - some strings are missing the i18n() around it<br>
>    - this does a bit to much in one line, I think: dirty? ++m_changeCounter :<br>
>    m_oldChangeCounter = m_changeCounter;<br>
><br>
>    As this is a Calligra wide change, it should go through reviewboard.<br>
<br>
</div>Thank you for the reply. I will post an amended patch there, then.<br>
<br>
_______________________________________________<br>
Krita mailing list<br>
<a href="mailto:kimageshop@kde.org">kimageshop@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kimageshop" target="_blank">https://mail.kde.org/mailman/listinfo/kimageshop</a><br>
</blockquote></div><br></div>