sven.langkamp at gmail.com
Sat Jun 4 21:12:53 CEST 2011
On Sat, Jun 4, 2011 at 3:34 PM, Silvio Heinrich <plassy at web.de> wrote:
> On 06/04/2011 05:09 AM, Sven Langkamp wrote:
> > Nice work. I noticed a few things:
> > -the old listview had a fix to have equal height for all the paintop
> > options, that should be included again
> They all should have an equal height, no?
> Or do you mean that the headers should have the same height as the options?
> I can do this.
There are items with and without checkbox and all should have the same
height. Header should also have the same height, I think.
> > -the collapse arrows should follow the KDE standard. So when collapsed
> > it should point right and down when expanded.
> Ok, can do this, too :D
> > -the presets don't stay selected after selecting them (not sure if
> > that is related to the branch)
> Hmm... I don't know too.
> The selection is normally handled in the preset chooser widget, am I right?
> Do you have a guess what could cause the deselection?
Yes, the preset chooser does that. No idea why it happens, usually it
shouldn't be related.
> > -opacity slider should go for 0 to 100
> All opacity sliders should be replaced with a KisDoubleSpinBox and the
> precision should be raised to at leased 3 digits after the comma.
> The reason is to have more fine control of the opacity.
> We internally setting the opacity with an unsigned 8bit integer that has
> a value range from 0 - 255.
> And we have those opacity sliders that will clamp the range to 0 - 100.
> We are loosing more then half of the precision for controlling the
> opacity here.
> This is why I replaced most of the sliders with double value versions.
> And I will raise the precision in the next few days :)
This isn't a matter of precision. If you have more digest before the comman,
you need less after the comma.
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the kimageshop