<div class="gmail_quote">On Mon, Jul 6, 2009 at 8:12 AM, Boudewijn Rempt <span dir="ltr">&lt;<a href="mailto:boud@valdyas.org">boud@valdyas.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Monday 06 July 2009, Sven Langkamp wrote:<br>
&gt;<br>
&gt;&gt;&gt;From what I can see the following features/bugs are left:<br>
&gt;<br>
&gt; -preset chooser/collection<br>
&gt; I&#39;m not sure how Boud had envisioned it. At the moment there is the<br>
&gt; combobox that only show &quot;default&quot; and the preset collection. Both do more<br>
&gt; or less nothing at moment.<br>
<br>
</div>Weird... the preset collection tab now shows patterns!<br>
</blockquote><div> <br>Yes, I currently have hard-coded the pattern resource server in the chooser.<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

But ok, I thought I had written down this before:<br>
<br>
* the second combobox and the preset collection will show exactly the same<br>
thing: all saved and pre-installed paintop presets for a particular brush<br>
engine. Those presets can be selected using either widget, but the collection<br>
tab will allow adding/removing presets. (Just as with other resources, like<br>
gradients).<br>
</blockquote><div><br>Maybe use the resource selector combobox that is already used by the flake pattern and gradient tools. It could show a preview of the current preset and have a list of available presets.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

What is missing here is the implementation of the save button in the &quot;Current<br>
Preset Definition&quot; tab. (the label in that tab should also contain a live<br>
preview of the preset with the current settings, like the one in the toolbar).<br>
<div class="im"><br>
&gt; How should they work and do we need both? For the delegate a preview of the<br>
&gt; paintop is needed. I guess it should look similar to the current brush<br>
&gt; preview.<br>
<br>
</div>We really need to be precise here, otherwise we&#39;ll get confused ourselves. The<br>
brush preview (for instance in the brush selection tab in the &quot;current preset<br>
definition&quot; pane) isn&#39;t suitable, and the preset preview in the toolbar may<br>
not be suitable either, because it needs a fairly6 large space. I was thinking<br>
of doing it similar to photoshop, and have the preset collection tab show<br>
either a single paintAt() preview in an icon view, or a listview of stroke<br>
previews with the name.<br><div class="im"></div></blockquote><div><br>Yeah, I meant the preset preview. It currently has the problem that lots of brushes can&#39;t show a useful preview.<br>Some brushes aren&#39;t suitable especially the bigger ones and paintops like deform are totally different. We will probably need special cases for some of the more special paintops.<br>
Maybe the artists can make some mockups how each paintop should look like? I think this is the most tricky part.<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
&gt; -settings widgets need to be per view<br>
&gt; <a href="https://bugs.kde.org/show_bug.cgi?id=174356" target="_blank">https://bugs.kde.org/show_bug.cgi?id=174356</a><br>
<br>
</div>I&#39;m fine with postponing that to the next release, if the saving/loading and<br>
selecting of presets gets in for 2.1. It sucks, but it is also quite a big<br>
operation.<br>
<div class="im"><br>
&gt; -nested boxes in the settings<br>
&gt; The settings widgets get a bit overstructured. Here are for example nested<br>
&gt; three tabwidgets:  <a href="http://img41.imageshack.us/img41/8729/paintopbox.png" target="_blank">http://img41.imageshack.us/img41/8729/paintopbox.png</a><br>
&gt; We should try to reduce the amount of borders.<br>
<br>
</div>I cannot take a look at that now, I&#39;m on the train. But in general, I&#39;m not<br>
concerned about over structured settings widgets for paintop presets yet --<br>
it&#39;s nothing compared to any other paint app yet :-)<br><div class="im"></div></blockquote><div><br>Have a look. The design of other apps is no excuse ;)<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">
&gt; Is there currently someone working on these issues? Boud? If not I will put<br>
&gt; these on my feature plan. Unfortunately all my exams will be prior to the<br>
&gt; hard-freeze, so I don&#39;t know how much I can get done.<br>
&gt; Of course code from any private git branch is welcome ;)<br>
<br>
</div>I&#39;ve been working on performance mostly, and some koffice library work. (And<br>
some work on making filters multithreaded again, and looking into all the<br>
regressions in our unittests) I haven&#39;t touched the preset<br>
code for quite some time, and I would love some help getting the<br>
loading/saving of presets as resources working.<br>
<font color="#888888"><br>
--<br>
Boudewijn Rempt | <a href="http://www.valdyas.org" target="_blank">http://www.valdyas.org</a><br>
<br>
_______________________________________________<br>
kimageshop 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>
</font></blockquote></div><br>