From: Joel Holdsworth Date: Thu, 1 Nov 2012 23:08:45 +0000 (+0000) Subject: Initial code to render text inside cursor markers X-Git-Tag: pulseview-0.1.0~226 X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=commitdiff_plain;h=34c1ef0254e1550c256c796e131a330d6ba8d81d;ds=sidebyside Initial code to render text inside cursor markers --- diff --git a/pv/signal.cpp b/pv/signal.cpp index c23a3954..bcb32813 100644 --- a/pv/signal.cpp +++ b/pv/signal.cpp @@ -21,6 +21,7 @@ #include #include "signal.h" +#include "view/view.h" namespace pv { diff --git a/pv/view/cursor.cpp b/pv/view/cursor.cpp index b854f2c4..35c2ed45 100644 --- a/pv/view/cursor.cpp +++ b/pv/view/cursor.cpp @@ -24,9 +24,14 @@ #include #include +#include #include #include +#include + +#include + namespace pv { namespace view { @@ -38,6 +43,8 @@ const QColor Cursor::TextColour(Qt::white); const int Cursor::Size = 12; const int Cursor::Offset = 1; +const int Cursor::ArrowSize = 4; + Cursor::Cursor(const View &view, double time) : TimeMarker(view, LineColour, time) { @@ -46,21 +53,70 @@ Cursor::Cursor(const View &view, double time) : QRectF Cursor::get_label_rect(const QRect &rect) const { const float x = (_time - _view.offset()) / _view.scale(); - return QRectF(x - Size/2, rect.height() - Size - Offset, - Size, Size); + + const QSizeF label_size( + _text_size.width() + View::LabelPadding.width() * 2, + _text_size.height() + View::LabelPadding.height() * 2); + return QRectF(x - label_size.width() / 2 - 0.5f, + rect.height() - label_size.height() - Offset - ArrowSize - 0.5f, + label_size.width() + 1, label_size.height() + 1); } void Cursor::paint_label(QPainter &p, const QRect &rect) { + compute_text_size(p); const QRectF r(get_label_rect(rect)); - p.setPen(LineColour); - p.setBrush(QBrush(FillColour)); - p.drawEllipse(r); + const float h_centre = (r.left() + r.right()) / 2; + const QPointF points[] = { + r.topRight(), + QPointF(r.right(), r.bottom()), + QPointF(h_centre + ArrowSize, r.bottom()), + QPointF(h_centre, rect.bottom()), + QPointF(h_centre - ArrowSize, r.bottom()), + QPointF(r.left(), r.bottom()), + r.topLeft() + }; + + const QPointF highlight_points[] = { + QPointF(r.right() - 1, r.top() + 1), + QPointF(r.right() - 1, r.bottom() - 1), + QPointF(h_centre + ArrowSize - 1, r.bottom() - 1), + QPointF(h_centre, rect.bottom() - 1), + QPointF(h_centre - ArrowSize + 1, r.bottom() - 1), + QPointF(r.left() + 1, r.bottom() - 1), + QPointF(r.left() + 1, r.top() + 1), + }; + + char text[16]; + format_text(text); + + p.setPen(Qt::transparent); + p.setBrush(FillColour); + p.drawPolygon(points, countof(points)); p.setPen(HighlightColour); - p.setBrush(QBrush()); - p.drawEllipse(r.adjusted(1, 1, -1, -1)); + p.setBrush(Qt::transparent); + p.drawPolygon(highlight_points, countof(highlight_points)); + + p.setPen(LineColour); + p.setBrush(Qt::transparent); + p.drawPolygon(points, countof(points)); + + p.setPen(TextColour); + p.drawText(r, Qt::AlignCenter | Qt::AlignVCenter, text); +} + +void Cursor::compute_text_size(QPainter &p) +{ + char text[16]; + format_text(text); + _text_size = p.boundingRect(QRectF(), 0, text).size(); +} + +void Cursor::format_text(char *text) +{ + sprintf(text, "%gs", _time); } } // namespace view diff --git a/pv/view/cursor.h b/pv/view/cursor.h index 3ee02005..1469f2cc 100644 --- a/pv/view/cursor.h +++ b/pv/view/cursor.h @@ -23,6 +23,10 @@ #include "timemarker.h" +#include + +class QPainter; + namespace pv { namespace view { @@ -34,10 +38,13 @@ private: static const QColor LineColour; static const QColor FillColour; static const QColor HighlightColour; + static const QColor TextColour; static const int Size; static const int Offset; + static const int ArrowSize; + public: /** * Constructor. @@ -60,6 +67,14 @@ public: * @param rect The rectangle of the ruler client area. */ void paint_label(QPainter &p, const QRect &rect); + +private: + void compute_text_size(QPainter &p); + + void format_text(char *text); + +private: + QSizeF _text_size; }; } // namespace view diff --git a/pv/view/timemarker.h b/pv/view/timemarker.h index 08ed9389..e8dac5d4 100644 --- a/pv/view/timemarker.h +++ b/pv/view/timemarker.h @@ -91,6 +91,8 @@ protected: const QColor &_colour; double _time; + + QSizeF _text_size; }; } // namespace view