KisSliderSpinBox api change

Boudewijn Rempt boud at valdyas.org
Sat Mar 13 09:23:39 CET 2010


On Saturday 13 March 2010, Sven Langkamp wrote:
> Hi,
> 
> when looking at the recent changes to use the new slider I found several
> places where is wasn't used correctly. (using the integer methods that does
> consider the factor)
> Therefor I propose the api changed in the attached patch. By not longer
> inheriting from QAbstractSlider it hides the internal api.
> 
> That way we can still use the current value/setValue and valueChanged
> signal.
> 
> When changing a slider the signal connection needs to be checked, because
> the old slider often had different parameter there.
> I know it's a bit stupid to change it now, but it reduces the risk of using
> the api in the wrong way.

Does this make it impossible to use the slider for integer values? We have a 
number of places where we use integers, like opacity etc.

-- 
Boudewijn Rempt | http://www.valdyas.org


More information about the kimageshop mailing list