<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"></div></blockquote><div><br>There are items with and without checkbox and all should have the same height. Header should also have the same height, I think.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">
><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><div class="im"></div></blockquote><div><br>Yes, the preset chooser does that. No idea why it happens, usually it shouldn't be related.<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">
><br>
> -opacity slider should go for 0 to 100<br>
</div>All opacity sliders should be replaced with a KisDoubleSpinBox and the<br>
precision should be raised to at leased 3 digits after the comma.<br>
The reason is to have more fine control of the opacity.<br>
We internally setting the opacity with an unsigned 8bit integer that has<br>
a value range from 0 - 255.<br>
And we have those opacity sliders that will clamp the range to 0 - 100.<br>
We are loosing more then half of the precision for controlling the<br>
opacity here.<br>
This is why I replaced most of the sliders with double value versions.<br>
And I will raise the precision in the next few days :)<br></blockquote></div><br>This isn't a matter of precision. If you have more digest before the comman, you need less after the comma.<br>With range 0-1 you would need a value 0.995 which would be 99.5 with range 0-100. I think that the latter is what most no scientist people are used to.<br>