Animation speed is a value, not a boolean

This breaks compatibility with old preferences, but it's a single
key and not that very important so I don't think it's a bigger issue
I've renamed prefs.animation to prefs.animation_speed to denote
that it's a value, and not a state.

Also, fixed the places that were treating it as a state (on/off)
to treat it like a correct value.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

2 files changed, 11 insertions, 11 deletions

diff --git a/qt-ui/profile/animationfunctions.cpp b/qt-ui/profile/animationfunctions.cpp
index 9f3aa8366..dc46d7f82 100644
--- a/qt-ui/profile/animationfunctions.cpp
+++ b/qt-ui/profile/animationfunctions.cpp
@@ -8,7 +8,7 @@ namespace Animations {
 
 	void hide(QObject *obj)
 	{
-		if (prefs.animation != 0) {
+		if (prefs.animation_speed != 0) {
 			QPropertyAnimation *animation = new QPropertyAnimation(obj, "opacity");
 			animation->setStartValue(1);
 			animation->setEndValue(0);
@@ -20,7 +20,7 @@ namespace Animations {
 
 	void animDelete(QObject *obj)
 	{
-		if (prefs.animation != 0) {
+		if (prefs.animation_speed != 0) {
 			QPropertyAnimation *animation = new QPropertyAnimation(obj, "opacity");
 			obj->connect(animation, SIGNAL(finished()), SLOT(deleteLater()));
 			animation->setStartValue(1);
@@ -33,9 +33,9 @@ namespace Animations {
 
 	void moveTo(QObject *obj, qreal x, qreal y)
 	{
-		if (prefs.animation != 0) {
+		if (prefs.animation_speed != 0) {
 			QPropertyAnimation *animation = new QPropertyAnimation(obj, "pos");
-			animation->setDuration(prefs.animation);
+			animation->setDuration(prefs.animation_speed);
 			animation->setStartValue(obj->property("pos").toPointF());
 			animation->setEndValue(QPointF(x, y));
 			animation->start(QAbstractAnimation::DeleteWhenStopped);
@@ -46,9 +46,9 @@ namespace Animations {
 
 	void scaleTo(QObject *obj, qreal scale)
 	{
-		if (prefs.animation != 0) {
+		if (prefs.animation_speed != 0) {
 			QPropertyAnimation *animation = new QPropertyAnimation(obj, "scale");
-			animation->setDuration(prefs.animation);
+			animation->setDuration(prefs.animation_speed);
 			animation->setStartValue(obj->property("scale").toReal());
 			animation->setEndValue(QVariant::fromValue(scale));
 			animation->setEasingCurve(QEasingCurve::InCubic);
diff --git a/qt-ui/profile/profilewidget2.cpp b/qt-ui/profile/profilewidget2.cpp
index 5e9a1d8c5..a6b3564f8 100644
--- a/qt-ui/profile/profilewidget2.cpp
+++ b/qt-ui/profile/profilewidget2.cpp
@@ -386,10 +386,10 @@ void ProfileWidget2::plotDive(struct dive *d, bool force)
 	}
 
 	// special handling for the first time we display things
-	int animSpeedBackup = -1;
+	int animSpeedBackup = 0;
 	if (firstCall && MainWindow::instance()->filesFromCommandLine()) {
-		animSpeedBackup = prefs.animation;
-		prefs.animation = 0;
+		animSpeedBackup = prefs.animation_speed;
+		prefs.animation_speed = 0;
 		firstCall = false;
 	}
 
@@ -518,8 +518,8 @@ void ProfileWidget2::plotDive(struct dive *d, bool force)
 	if ((nr = number_of_computers(&displayed_dive)) > 1)
 		dcText += tr(" (#%1 of %2)").arg(dc_number + 1).arg(nr);
 	diveComputerText->setText(dcText);
-	if (MainWindow::instance()->filesFromCommandLine() && animSpeedBackup != -1) {
-		prefs.animation = animSpeedBackup;
+	if (MainWindow::instance()->filesFromCommandLine() && animSpeedBackup != 0) {
+		prefs.animation_speed = animSpeedBackup;
 	}
 
 	if (currentState == ADD || currentState == PLAN) { // TODO: figure a way to move this from here.