<div dir="ltr">On Tue, May 14, 2013 at 5:13 PM,  <span dir="ltr"><<a href="mailto:s_suelzer@lavabit.com" target="_blank">s_suelzer@lavabit.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hi. :)<br>
<br>
I've attached a patch that improves the way the user can organize<br>
resources into named sets that can be added, deleted and chosen from a<br>
drop down menu that I've added to KoResourceItemChooser. This made the<br>
changes available to the paintoppreset choosers, the gradient choosers<br>
and the pattern choosers.<br>
<br>
Currenly only adding and deleting categories is supported, renaming them<br>
is in the code but lacks an UI feature, which I was unable to implement.<br>
(I wanted the unfiltered view to be immutable and tried dis and enabling<br>
the editable state of the drop down box based on which index was active,<br>
but that introduced a segfault.)<br>
<br>
I have removed the `show all' checkbox in the paintop preset docker and<br>
all references to that functionality since it made filtering a bit more<br>
complicated as well as just not making too much sense with the<br>
improvements to the tagging system.<br>
<br>
I have also removed the proxy adapter in KisPresetChooser since it<br>
didn't do anything useful after these changes, yet required attention<br>
since it overloaded some now improved functions.<br>
<br>
Current shortcomings of the system:<br>
Since I used the already existing KoResourceTagging class to handle all<br>
the tagging bits, and ultimaltely save tags to disk, in alphabetical<br>
order, restarting Krita, and thus repopulating the combo boxes also<br>
fills them in alphabetical order and not in the order they might have<br>
been defined by the user.<br>
<br>
I also had a crash at one point in time where some brush presets got<br>
changed around in master and clicking some resources in the brush preset<br>
box crashed the program, but I was not able to reproduce that after a<br>
clean rebuild of my local workspace and changing presets around<br>
externally, so it might not be an issue, but for completeness sake ;).<br>
<br>
And that's it.<br>
Cheers~!<br></blockquote><div><br></div><div style>Overall pretty nice work.</div><div style><br></div><div style>There are a few minor things I spotted:</div><div style><br></div><div style>- that empty slot in PresetDockerDock can be removed completely including the connections</div>
<div style>- there were some coding styles issues with missing spaces between methods <br></div><div style>- KoPresetFiltering has a misleading name, at that level we don't have presets just resources</div><div style>
- some strings are missing the i18n() around it</div><div style>- this does a bit to much in one line, I think: dirty? ++m_changeCounter : m_oldChangeCounter = m_changeCounter;</div><div> </div><div style>As this is a Calligra wide change, it should go through reviewboard.</div>
</div></div></div>