X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fview%2Fruler.cpp;h=4649b10d69648ddfc4625acb8a94ef01b8d9a32c;hp=83863e61ea203e3a3efb08f772425a8510da029d;hb=f260e3bfe8cc0f221b3d4498b548295b9168ffbf;hpb=f76af6375b8aea6b7edb2d6ee838e1589c3490f3

diff --git a/pv/view/ruler.cpp b/pv/view/ruler.cpp
index 83863e61..4649b10d 100644
--- a/pv/view/ruler.cpp
+++ b/pv/view/ruler.cpp
@@ -19,6 +19,8 @@
  */
 
 #include "ruler.h"
+
+#include "cursor.h"
 #include "view.h"
 #include "viewport.h"
 
@@ -26,6 +28,7 @@
 
 #include <assert.h>
 #include <math.h>
+#include <limits.h>
 
 #include <QMouseEvent>
 #include <QPainter>
@@ -47,7 +50,8 @@ const int Ruler::HoverArrowSize = 5;
 
 Ruler::Ruler(View &parent) :
 	QWidget(&parent),
-	_view(parent)
+	_view(parent),
+	_grabbed_marker(NULL)
 {
 	setMouseTracking(true);
 
@@ -55,37 +59,56 @@ Ruler::Ruler(View &parent) :
 		this, SLOT(hover_point_changed()));
 }
 
-void Ruler::paintEvent(QPaintEvent *event)
+void Ruler::paintEvent(QPaintEvent*)
 {
+	using namespace Qt;
+
+	const double SpacingIncrement = 32.0f;
+	const double MinValueSpacing = 32.0f;
+	const int ValueMargin = 3;
+
 	QPainter p(this);
 	p.setRenderHint(QPainter::Antialiasing);
 
-	const double MinSpacing = 80;
+	double min_width = SpacingIncrement, typical_width;
+	double tick_period;
+	unsigned int prefix;
+	double multiplier;
 
-	const double min_period = _view.scale() * MinSpacing;
+	// Find tick spacing, and number formatting that does not cause
+	// value to collide.
+	do
+	{
+		const double min_period = _view.scale() * min_width;
 
-	const int order = (int)floorf(log10f(min_period));
-	const double order_decimal = pow(10, order);
+		const int order = (int)floorf(log10f(min_period));
+		const double order_decimal = pow(10, order);
 
-	unsigned int unit = 0;
-	double tick_period = 0.0f;
+		unsigned int unit = 0;
 
-	do
-	{
-		tick_period = order_decimal * ScaleUnits[unit++];
-	} while(tick_period < min_period && unit < countof(ScaleUnits));
+		do
+		{
+			tick_period = order_decimal * ScaleUnits[unit++];
+		} while (tick_period < min_period && unit < countof(ScaleUnits));
 
-	const unsigned int prefix = (order - FirstSIPrefixPower) / 3;
-	assert(prefix >= 0);
-	assert(prefix < countof(SIPrefixes));
+		prefix = (order - FirstSIPrefixPower) / 3;
+		assert(prefix < countof(SIPrefixes));
 
-	const double multiplier = pow(10.0, - prefix * 3 - FirstSIPrefixPower);
+		multiplier = pow(10.0, - prefix * 3 - FirstSIPrefixPower);
+
+		typical_width = p.boundingRect(0, 0, INT_MAX, INT_MAX,
+			AlignLeft | AlignTop, format_time(_view.offset(),
+			multiplier, prefix)).width() + MinValueSpacing;
+
+		min_width += SpacingIncrement;
+
+	} while(typical_width > tick_period / _view.scale());
 
 	const int text_height = p.boundingRect(0, 0, INT_MAX, INT_MAX,
-		Qt::AlignLeft | Qt::AlignTop, "8").height();
+		AlignLeft | AlignTop, "8").height();
 
 	// Draw the tick marks
-	p.setPen(Qt::black);
+	p.setPen(palette().color(foregroundRole()));
 
 	const double minor_tick_period = tick_period / MinorTickSubdivision;
 	const double first_major_division =
@@ -96,30 +119,33 @@ void Ruler::paintEvent(QPaintEvent *event)
 
 	int division = (int)round(first_minor_division -
 		first_major_division * MinorTickSubdivision);
-	while(1)
+
+	const int major_tick_y1 = text_height + ValueMargin * 2;
+	const int tick_y2 = height();
+	const int minor_tick_y1 = (major_tick_y1 + tick_y2) / 2;
+
+	while (1)
 	{
 		const double t = t0 + division * minor_tick_period;
 		const double x = (t - _view.offset()) / _view.scale();
 
-		if(x >= width())
+		if (x >= width())
 			break;
 
-		if(division % MinorTickSubdivision == 0)
+		if (division % MinorTickSubdivision == 0)
 		{
 			// Draw a major tick
-			QString s;
-			QTextStream ts(&s);
-			ts << (t * multiplier) << SIPrefixes[prefix] << "s";
-			p.drawText(x, 0, 0, text_height, Qt::AlignCenter |
-				Qt::AlignTop | Qt::TextDontClip, s);
-			p.drawLine(QPointF(x, text_height),
-				QPointF(x, height()));
+			p.drawText(x, ValueMargin, 0, text_height,
+				AlignCenter | AlignTop | TextDontClip,
+				format_time(t, multiplier, prefix));
+			p.drawLine(QPointF(x, major_tick_y1),
+				QPointF(x, tick_y2));
 		}
 		else
 		{
 			// Draw a minor tick
-			p.drawLine(QPointF(x, (text_height + height()) / 2),
-				QPointF(x, height()));
+			p.drawLine(QPointF(x, minor_tick_y1),
+				QPointF(x, tick_y2));
 		}
 
 		division++;
@@ -134,9 +160,52 @@ void Ruler::paintEvent(QPaintEvent *event)
 	p.end();
 }
 
+void Ruler::mouseMoveEvent(QMouseEvent *e)
+{
+	if (!_grabbed_marker)
+		return;
+
+	_grabbed_marker->set_time(_view.offset() +
+		((double)e->x() + 0.5) * _view.scale());
+}
+
+void Ruler::mousePressEvent(QMouseEvent *e)
+{
+	if (e->buttons() & Qt::LeftButton) {
+		_grabbed_marker = NULL;
+
+		if (_view.cursors_shown()) {
+			std::pair<Cursor, Cursor> &cursors =
+				_view.cursors();
+			if (cursors.first.get_label_rect(
+				rect()).contains(e->pos()))
+				_grabbed_marker = &cursors.first;
+			else if (cursors.second.get_label_rect(
+				rect()).contains(e->pos()))
+				_grabbed_marker = &cursors.second;
+		}
+	}
+}
+
+void Ruler::mouseReleaseEvent(QMouseEvent *)
+{
+	_grabbed_marker = NULL;
+}
+
+QString Ruler::format_time(double t, double multiplier,
+	unsigned int prefix)
+{
+	QString s;
+	QTextStream ts(&s);
+	ts.setRealNumberPrecision(0);
+	ts << fixed << forcesign << (t  * multiplier) <<
+		SIPrefixes[prefix] << "s";
+	return s;
+}
+
 void Ruler::draw_cursors(QPainter &p)
 {
-	if(!_view.cursors_shown())
+	if (!_view.cursors_shown())
 		return;
 
 	const QRect r = rect();
@@ -148,7 +217,8 @@ void Ruler::draw_cursors(QPainter &p)
 void Ruler::draw_hover_mark(QPainter &p)
 {
 	const int x = _view.hover_point().x();
-	if(x == -1)
+
+	if (x == -1 || _grabbed_marker)
 		return;
 
 	p.setPen(QPen(Qt::NoPen));