Fix the positioning of the Labels using the new DiveTextItem

This uses a combination of items on the canvas which makes it easier to
position it where I want.

This also broke the other texts because I forgot about them. I will
fix that on the next commit.

Signed-off-by: Tomaz Canabrava <tcanabrava@kde.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

qt-ui/profile/divecartesianaxis.cpp

@@ -27,7 +27,7 @@ void DiveCartesianAxis::setTextColor(const QColor& color)
 	textColor = color;
 }
 
-DiveCartesianAxis::DiveCartesianAxis() : orientation(LeftToRight)
+DiveCartesianAxis::DiveCartesianAxis() : orientation(LeftToRight), showTicks(true), showText(true)
 {
 
 }
@@ -50,16 +50,30 @@ void DiveCartesianAxis::updateTicks()
 	double steps = (max - min) / interval;
 	double currValue = min;
 
+	if (!showTicks && !ticks.empty()){
+		qDeleteAll(ticks);
+		ticks.clear();
+	}
+
+	if(!showText && !labels.empty()){
+			qDeleteAll(labels);
+			labels.clear();
+	}
+
 	// Remove the uneeded Ticks / Texts.
-	if (!ticks.isEmpty() && ticks.size() > steps) {
+	if (showTicks && !ticks.isEmpty() && ticks.size() > steps) {
 		while (ticks.size() > steps) {
 			DiveLineItem *removedLine = ticks.takeLast();
 			removedLine->animatedHide();
+		}
+	}
+
+	if (!labels.isEmpty() && labels.size() > steps) {
+		while (labels.size() > steps) {
 				DiveTextItem *removedText = labels.takeLast();
 				removedText->animatedHide();
 		}
 	}
-
 	// Move the remaining Ticks / Text to it's corerct position
 	// Regartind the possibly new values for the Axis
 	qreal begin, stepSize;
@@ -78,7 +92,6 @@ void DiveCartesianAxis::updateTicks()
 	}
 	stepSize = stepSize / steps;
 
-
 	for (int i = 0, count = ticks.size(); i < count; i++, currValue += interval) {
 		qreal childPos;
 		if (orientation == TopToBottom || orientation == LeftToRight) {
@@ -104,32 +117,57 @@ void DiveCartesianAxis::updateTicks()
 		} else {
 			childPos = begin - i * stepSize;
 		}
-		DiveLineItem *item = new DiveLineItem(this);
+		DiveLineItem *item = NULL;
+		DiveTextItem *label = NULL;
+
+		if (showTicks){
+			item = new DiveLineItem(this);
 			item->setPen(pen());
 			ticks.push_back(item);
-
+		}
-		DiveTextItem *label = new DiveTextItem(this);
+		if (showText){
+			label = new DiveTextItem(this);
 			label->setText(textForValue(currValue));
 			label->setBrush(QBrush(textColor));
-
+		}
 		labels.push_back(label);
 		if (orientation == RightToLeft || orientation == LeftToRight) {
+			if (showTicks){
 				item->setLine(0, 0, 0, tickSize);
 				item->setPos(scene()->sceneRect().width() + 10, m.y1() + tickSize); // position it outside of the scene
 				item->animateMoveTo(childPos, m.y1() + tickSize); // anim it to scene.
+			}
+			if(showText){
 				label->setAlignment(Qt::AlignBottom | Qt::AlignHCenter);
 				label->setPos(scene()->sceneRect().width() + 10, m.y1() + tickSize); // position it outside of the scene);
 				label->animateMoveTo(childPos, m.y1() + tickSize);
+			}
 		} else {
+			if(showTicks){
 				item->setLine(0, 0, tickSize, 0);
 				item->setPos(m.x1() - tickSize, scene()->sceneRect().height() + 10);
 				item->animateMoveTo(m.x1() - tickSize, childPos);
+			}
+			if(showText){
 				label->setAlignment(Qt::AlignVCenter| Qt::AlignRight);
 				label->setPos(m.x1() - tickSize, scene()->sceneRect().height() + 10);
 				label->animateMoveTo(m.x1() - tickSize, childPos);
 			}
 		}
 	}
+}
+
+void DiveCartesianAxis::setShowText(bool show)
+{
+	showText = show;
+	updateTicks();
+}
+
+void DiveCartesianAxis::setShowTicks(bool show)
+{
+	showTicks = show;
+	updateTicks();
+}
 
 QString DiveCartesianAxis::textForValue(double value)
 {

qt-ui/profile/divecartesianaxis.h

@@ -29,6 +29,8 @@ public:
 	qreal posAtValue(qreal value);
 	void setColor(const QColor& color);
 	void setTextColor(const QColor& color);
+	void setShowTicks(bool show);
+	void setShowText(bool show);
 	int unitSystem;
 signals:
 	void sizeChanged();
@@ -43,6 +45,8 @@ protected:
 	double interval;
 	double tickSize;
 	QColor textColor;
+	bool showTicks;
+	bool showText;
 };
 
 class DepthAxis : public DiveCartesianAxis {

qt-ui/profile/divetextitem.cpp

@@ -7,6 +7,7 @@
 #include <QFontMetrics>
 #include <QBrush>
 #include <QPen>
+#include <QDebug>
 
 DiveTextItem::DiveTextItem(QGraphicsItem* parent): QGraphicsItemGroup(parent),
 	textBackgroundItem(NULL),
@@ -50,10 +51,10 @@ void DiveTextItem::updateText()
 
 	QRectF rect = fm.boundingRect(text);
 	yPos = (internalAlignFlags & Qt::AlignTop) ? -rect.height() :
-			(internalAlignFlags & Qt::AlignBottom) ? 0 :
+			(internalAlignFlags & Qt::AlignBottom) ? +rect.height() :
-	/*(internalAlignFlags & Qt::AlignVCenter  ? */ -rect.height() / 2;
+	/*(internalAlignFlags & Qt::AlignVCenter  ? */ +rect.height() / 4;
 
-	yPos = (internalAlignFlags & Qt::AlignLeft ) ? 0 :
+	xPos = (internalAlignFlags & Qt::AlignLeft ) ? +rect.width() :
 		(internalAlignFlags & Qt::AlignHCenter) ?  -rect.width()/2 :
 	 /* (internalAlignFlags & Qt::AlignRight) */ -rect.width();
 

qt-ui/profile/profilewidget2.cpp

@@ -294,12 +294,10 @@ void ProfileWidget2::plotDives(QList<dive*> dives)
 	profileYAxis->updateTicks();
 	temperatureAxis->setMinimum(pInfo.mintemp);
 	temperatureAxis->setMaximum(pInfo.maxtemp);
-	//temperatureAxis->updateTicks();
 	timeAxis->setMaximum(maxtime);
 	timeAxis->updateTicks();
 	cylinderPressureAxis->setMinimum(pInfo.minpressure);
 	cylinderPressureAxis->setMaximum(pInfo.maxpressure);
-	cylinderPressureAxis->updateTicks();
 	meanDepth->animateMoveTo(3, profileYAxis->posAtValue(pInfo.meandepth));
 	dataModel->setDive(current_dive, pInfo);
 

qt-ui/profilegraphics.cpp

@@ -362,6 +362,7 @@ QColor ProfileGraphicsView::getColor(const color_indice_t i)
 
 void ProfileGraphicsView::plot(struct dive *d, bool forceRedraw)
 {
+	return;
 	struct divecomputer *dc = NULL;
 
 	if (d)