Review Request 121085: Migrate ProgressBar to QtQuick.Controls
Kai Uwe Broulik
kde at privat.broulik.de
Mon Nov 10 21:11:24 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121085/
-----------------------------------------------------------
(Updated Nov. 10, 2014, 9:11 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma.
Repository: plasma-framework
Description
-------
This also fixes a regression introduced in 7a5342e97067cffb316fd57e20f1ca326a0854ee where the devicePixelRatio was no longer taken into account.
The test is enhanced to also cover vertical bars. The QtQuick.Controls ProgressBar just rotates the progress bar by 90° when vertical so I could remove a lot of the (vertical ? foo : bar) everywhere.
I tried to migrate the indeterminate animation to Animators but after thinking about adding timers and having a dozen of bindings to reset the animation I stopped.
Diffs
-----
tests/components/progressbar.qml 7098650
src/declarativeimports/plasmacomponents/qml/styles/ProgressBarStyle.qml PRE-CREATION
src/declarativeimports/plasmacomponents/qml/ProgressBar.qml 69b6df1
Diff: https://git.reviewboard.kde.org/r/121085/diff/
Testing
-------
Battery monitor looks like it did before, used the progressbar test to spot regressions.
The only regression I found was that
minimumValue: -100
maximumValue: 100
value: 0
yields an empty 0% bar instead, value: 50 a 50%, seems minimumValue cannot go below 0?
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20141110/5643a4c0/attachment.html>
More information about the Plasma-devel
mailing list