X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fviews%2Ftrace%2Fcursorpair.cpp;h=562584c2f903fdde8f93b0bbd32005bc58c1673b;hp=9a8744054bb0792fafded3281bf8686a7fe26c50;hb=ef85cfa4599d3741e7007921e39a44490e97cfaf;hpb=1573bf16ba50d1c023ad3a9ce596f0ab6eaeacff
diff --git a/pv/views/trace/cursorpair.cpp b/pv/views/trace/cursorpair.cpp
index 9a874405..562584c2 100644
--- a/pv/views/trace/cursorpair.cpp
+++ b/pv/views/trace/cursorpair.cpp
@@ -17,15 +17,19 @@
* along with this program; if not, see .
*/
+#include
+#include
+
+#include
+#include
+
#include "cursorpair.hpp"
+#include "pv/globalsettings.hpp"
#include "pv/util.hpp"
#include "ruler.hpp"
#include "view.hpp"
-#include
-#include
-
using std::max;
using std::make_pair;
using std::min;
@@ -37,13 +41,25 @@ namespace views {
namespace trace {
const int CursorPair::DeltaPadding = 8;
-const QColor CursorPair::ViewportFillColour(220, 231, 243);
CursorPair::CursorPair(View &view) :
TimeItem(view),
first_(new Cursor(view, 0.0)),
second_(new Cursor(view, 1.0))
{
+ GlobalSettings::add_change_handler(this);
+
+ GlobalSettings settings;
+ fill_color_ = QColor::fromRgba(settings.value(
+ GlobalSettings::Key_View_CursorFillColor).value());
+
+ connect(&view_, SIGNAL(hover_point_changed(const QWidget*, QPoint)),
+ this, SLOT(on_hover_point_changed(const QWidget*, QPoint)));
+}
+
+CursorPair::~CursorPair()
+{
+ GlobalSettings::remove_change_handler(this);
}
bool CursorPair::enabled() const
@@ -73,9 +89,9 @@ float CursorPair::get_x() const
return (first_->get_x() + second_->get_x()) / 2.0f;
}
-QPoint CursorPair::point(const QRect &rect) const
+QPoint CursorPair::drag_point(const QRect &rect) const
{
- return first_->point(rect);
+ return first_->drag_point(rect);
}
pv::widgets::Popup* CursorPair::create_popup(QWidget *parent)
@@ -110,38 +126,36 @@ void CursorPair::paint_label(QPainter &p, const QRect &rect, bool hover)
if (!enabled())
return;
- const QColor text_colour =
- ViewItem::select_text_colour(Cursor::FillColour);
+ const QColor text_color = ViewItem::select_text_color(Cursor::FillColor);
+ p.setPen(text_color);
- p.setPen(text_colour);
- compute_text_size(p);
QRectF delta_rect(label_rect(rect));
-
const int radius = delta_rect.height() / 2;
- const QRectF text_rect(delta_rect.intersected(
- rect).adjusted(radius, 0, -radius, 0));
- if (text_rect.width() >= text_size_.width()) {
- const int highlight_radius = delta_rect.height() / 2 - 2;
-
- if (selected()) {
- p.setBrush(Qt::transparent);
- p.setPen(highlight_pen());
- p.drawRoundedRect(delta_rect, radius, radius);
- }
-
- p.setBrush(hover ? Cursor::FillColour.lighter() :
- Cursor::FillColour);
- p.setPen(Cursor::FillColour.darker());
- p.drawRoundedRect(delta_rect, radius, radius);
+ QRectF text_rect(delta_rect.intersected(rect).adjusted(radius, 0, -radius, 0));
- delta_rect.adjust(1, 1, -1, -1);
- p.setPen(Cursor::FillColour.lighter());
- p.drawRoundedRect(delta_rect, highlight_radius, highlight_radius);
+ QString text = format_string(text_rect.width(), [&p](const QString& s) -> qreal {
+ return p.boundingRect(QRectF(), 0, s).width();
+ });
+ text_size_ = p.boundingRect(QRectF(), 0, text).size();
- p.setPen(text_colour);
- p.drawText(text_rect, Qt::AlignCenter | Qt::AlignVCenter,
- format_string());
+ if (selected()) {
+ p.setBrush(Qt::transparent);
+ p.setPen(highlight_pen());
+ p.drawRoundedRect(delta_rect, radius, radius);
}
+
+ p.setBrush(hover ? Cursor::FillColor.lighter() : Cursor::FillColor);
+ p.setPen(Cursor::FillColor.darker());
+ p.drawRoundedRect(delta_rect, radius, radius);
+
+ delta_rect.adjust(1, 1, -1, -1);
+ p.setPen(Cursor::FillColor.lighter());
+ const int highlight_radius = delta_rect.height() / 2 - 2;
+ p.drawRoundedRect(delta_rect, highlight_radius, highlight_radius);
+ label_area_ = delta_rect;
+
+ p.setPen(text_color);
+ p.drawText(text_rect, Qt::AlignCenter | Qt::AlignVCenter, text);
}
void CursorPair::paint_back(QPainter &p, ViewItemPaintParams &pp)
@@ -150,36 +164,59 @@ void CursorPair::paint_back(QPainter &p, ViewItemPaintParams &pp)
return;
p.setPen(Qt::NoPen);
- p.setBrush(QBrush(ViewportFillColour));
+ p.setBrush(fill_color_);
const pair offsets(get_cursor_offsets());
- const int l = (int)max(min(
- offsets.first, offsets.second), 0.0f);
- const int r = (int)min(max(
- offsets.first, offsets.second), (float)pp.width());
+ const int l = (int)max(min(offsets.first, offsets.second), 0.0f);
+ const int r = (int)min(max(offsets.first, offsets.second), (float)pp.width());
p.drawRect(l, pp.top(), r - l, pp.height());
}
-QString CursorPair::format_string()
+QString CursorPair::format_string(qreal max_width, std::function query_size)
{
+ constexpr int time_precision = 12;
+ constexpr int freq_precision = 4;
+
const pv::util::SIPrefix prefix = view_.tick_prefix();
const pv::util::Timestamp diff = abs(second_->time() - first_->time());
- const QString s1 = Ruler::format_time_with_distance(
- diff, diff, prefix, view_.time_unit(), view_.tick_precision(), false);
- const QString s2 = util::format_time_si(
- 1 / diff, pv::util::SIPrefix::unspecified, 4, "Hz", false);
+ const QString time = Ruler::format_time_with_distance(
+ diff, diff, prefix, view_.time_unit(), time_precision, false);
+ const QString freq = util::format_time_si(
+ 1 / diff, pv::util::SIPrefix::unspecified, freq_precision, "Hz", false);
+ const QString out = QString("%1 / %2").arg(time, freq);
- return QString("%1 / %2").arg(s1).arg(s2);
-}
+ // Try full "{time} ms / {freq} Hz" format
+ if (max_width <= 0 || query_size(out) <= max_width) {
+ label_incomplete_ = false;
+ return out;
+ }
-void CursorPair::compute_text_size(QPainter &p)
-{
- assert(first_);
- assert(second_);
+ label_incomplete_ = true;
+
+ // Try just "{time}ms" format and gradually reduce time precision down to zero
+ for (int shrinkage=0; shrinkage <= time_precision; shrinkage++) {
+ int prec = time_precision - shrinkage ;
- text_size_ = p.boundingRect(QRectF(), 0, format_string()).size();
+ const QString time = Ruler::format_time_with_distance(
+ diff, diff, prefix, view_.time_unit(),
+ prec, false);
+
+ if (query_size(time) <= max_width)
+ return time;
+ }
+
+ // Try no trailing digits and drop the unit to at least display something. The unit should be obvious from the ruler
+ // anyway.
+ const QString bare_number = Ruler::format_time_with_distance(
+ diff, diff, prefix, view_.time_unit(),
+ 0, false, false);
+ if (query_size(bare_number) <= max_width)
+ return bare_number;
+
+ // Give up.
+ return "...";
}
pair CursorPair::get_cursor_offsets() const
@@ -190,6 +227,26 @@ pair CursorPair::get_cursor_offsets() const
return pair(first_->get_x(), second_->get_x());
}
+void CursorPair::on_setting_changed(const QString &key, const QVariant &value)
+{
+ if (key == GlobalSettings::Key_View_CursorFillColor)
+ fill_color_ = QColor::fromRgba(value.value());
+}
+
+void CursorPair::on_hover_point_changed(const QWidget* widget, const QPoint& hp)
+{
+ if (widget != view_.ruler())
+ return;
+
+ if (!label_incomplete_)
+ return;
+
+ if (label_area_.contains(hp))
+ QToolTip::showText(view_.mapToGlobal(hp), format_string());
+ else
+ QToolTip::hideText(); // TODO Will break other tooltips when there can be others
+}
+
} // namespace trace
} // namespace views
} // namespace pv