Review Request 122246: Add addSample(qreal value) overload

[Marco Martin]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122246/#review74754
-----------------------------------------------------------

Ship it!


Ship It!

- Marco Martin


On Jan. 25, 2015, 3:11 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122246/
> -----------------------------------------------------------
> 
> (Updated Jan. 25, 2015, 3:11 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> -------
> 
> Add an addSample(qreal value) overload making it easier to work with just a simple dataset. This way you can do plotter.addSample(50) instead of plotter.addSample([50]).
> 
> QML seems to accept single strings for QStringList but not other list types.
> 
> 
> Diffs
> -----
> 
>   src/qmlcontrols/kquickcontrolsaddons/plotter.h 3a10212 
>   src/qmlcontrols/kquickcontrolsaddons/plotter.cpp 65dee7a 
> 
> Diff: https://git.reviewboard.kde.org/r/122246/diff/
> 
> 
> Testing
> -------
> 
> addSample(50) and addSample([40, 41, 42]) work
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150126/5e89c0f3/attachment-0001.html>