Review Request: Add missing property alias "animation" in Slider Plasma Component

[David Edmundson]

> On March 18, 2012, 7:57 p.m., Aaron J. Seigo wrote:
> > what is the use case for knowing if it is animated or not? this seems like an internal implementation detail?

I assume it's more for setting whether it animates or not. (when the user clicks on the slider, whether it slides to the new value) Tbh, I'm with you that it's not needed. 

However the gallery is full of 
Slider {
....
 animated:true;
} 

and the documentation at the top of this file says:

bool animated
This property holds if the slider will animate or not when other point is clicked,
and the slider handler is not being dragged. The default value is false.

So it looked to me like it was meant to exist, and was missed off.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104318/#review11571
-----------------------------------------------------------


On March 17, 2012, 7:27 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104318/
> -----------------------------------------------------------
> 
> (Updated March 17, 2012, 7:27 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> Add missing property alias "animation" in Slider Plasma Component
> 
> Item now matches it's own documentation.
> 
> 
> Diffs
> -----
> 
>   plasma/declarativeimports/plasmacomponents/qml/Slider.qml 447dbb1 
> 
> Diff: http://git.reviewboard.kde.org/r/104318/diff/
> 
> 
> Testing
> -------
> 
> Tested in Component Gallery (which didn't work before!) 
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120318/6eaa0472/attachment.html>