<div class="gmail_quote">On Sat, Jun 4, 2011 at 3:34 PM, Silvio Heinrich <span dir="ltr"><<a href="mailto:plassy@web.de">plassy@web.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 06/04/2011 05:09 AM, Sven Langkamp wrote:<br>
><br>
> Nice work. I noticed a few things:<br>
><br>
> -the old listview had a fix to have equal height for all the paintop<br>
> options, that should be included again<br>
</div>They all should have an equal height, no?<br>
Or do you mean that the headers should have the same height as the options?<br>
I can do this.<br>
<div class="im"><br>
><br>
> -the collapse arrows should follow the KDE standard. So when collapsed<br>
> it should point right and down when expanded.<br>
</div>Ok, can do this, too :D<br>
<div class="im">><br>
> -the presets don't stay selected after selecting them (not sure if<br>
> that is related to the branch)<br>
</div>Hmm... I don't know too.<br>
The selection is normally handled in the preset chooser widget, am I right?<br>
Do you have a guess what could cause the deselection?<br></blockquote></div><br>Fixed. Now the presetchanged is emitted by the canvas resource provider which wasn't the case before and reset the chooser.<br>