X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=pv%2Fview%2Ftrace.cpp;h=e1b51373ae667616895f09f76c224283fd4f410c;hb=e0ddc2a408e3b667ca8e15d8482ecdcb03d46d30;hp=05054d4927001c260d271d713ba88bbc14433029;hpb=99af6802961b0a5ec7ec7d3ff2215c8eb6925ca3;p=pulseview.git
diff --git a/pv/view/trace.cpp b/pv/view/trace.cpp
index 05054d4..e1b5137 100644
--- a/pv/view/trace.cpp
+++ b/pv/view/trace.cpp
@@ -14,13 +14,12 @@
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ * along with this program; if not, see .
*/
#include
-#include
+#include
#include
#include
@@ -32,50 +31,39 @@
#include "tracepalette.hpp"
#include "view.hpp"
-#include
-#include
+#include "pv/globalsettings.hpp"
+#include "pv/widgets/colourbutton.hpp"
+#include "pv/widgets/popup.hpp"
+
+using std::pair;
+using std::shared_ptr;
namespace pv {
-namespace view {
+namespace views {
+namespace TraceView {
-const QPen Trace::AxisPen(QColor(128, 128, 128, 64));
+const QPen Trace::AxisPen(QColor(0, 0, 0, 30 * 256 / 100));
const int Trace::LabelHitPadding = 2;
-Trace::Trace(QString name) :
- name_(name),
+const QColor Trace::BrightGrayBGColour = QColor(0, 0, 0, 10 * 255 / 100);
+const QColor Trace::DarkGrayBGColour = QColor(0, 0, 0, 15 * 255 / 100);
+
+Trace::Trace(shared_ptr channel) :
+ base_(channel),
popup_(nullptr),
popup_form_(nullptr)
{
-}
-
-QString Trace::name() const
-{
- return name_;
-}
-
-void Trace::set_name(QString name)
-{
- name_ = name;
-}
-
-QColor Trace::colour() const
-{
- return colour_;
-}
-
-void Trace::set_colour(QColor colour)
-{
- colour_ = colour;
-
- bgcolour_ = colour;
- bgcolour_.setAlpha(20);
+ connect(channel.get(), SIGNAL(name_changed(const QString&)),
+ this, SLOT(on_name_changed(const QString&)));
+ connect(channel.get(), SIGNAL(colour_changed(const QColor&)),
+ this, SLOT(on_colour_changed(const QColor&)));
}
void Trace::paint_label(QPainter &p, const QRect &rect, bool hover)
{
const int y = get_visual_y();
- p.setBrush(colour_);
+ p.setBrush(base_->colour());
if (!enabled())
return;
@@ -106,23 +94,23 @@ void Trace::paint_label(QPainter &p, const QRect &rect, bool hover)
}
p.setPen(Qt::transparent);
- p.setBrush(hover ? colour_.lighter() : colour_);
+ p.setBrush(hover ? base_->colour().lighter() : base_->colour());
p.drawPolygon(points, countof(points));
- p.setPen(colour_.lighter());
+ p.setPen(base_->colour().lighter());
p.setBrush(Qt::transparent);
p.drawPolygon(highlight_points, countof(highlight_points));
- p.setPen(colour_.darker());
+ p.setPen(base_->colour().darker());
p.setBrush(Qt::transparent);
p.drawPolygon(points, countof(points));
// Paint the text
- p.setPen(select_text_colour(colour_));
+ p.setPen(select_text_colour(base_->colour()));
p.setFont(QApplication::font());
p.drawText(QRectF(r.x(), r.y(),
r.width() - label_arrow_length, r.height()),
- Qt::AlignCenter | Qt::AlignVCenter, name_);
+ Qt::AlignCenter | Qt::AlignVCenter, base_->name());
}
QMenu* Trace::create_context_menu(QWidget *parent)
@@ -142,19 +130,16 @@ pv::widgets::Popup* Trace::create_popup(QWidget *parent)
create_popup_form();
- connect(popup_, SIGNAL(closed()),
- this, SLOT(on_popup_closed()));
+ connect(popup_, SIGNAL(closed()), this, SLOT(on_popup_closed()));
return popup_;
}
QRectF Trace::label_rect(const QRectF &rect) const
{
- using pv::view::View;
-
QFontMetrics m(QApplication::font());
const QSize text_size(
- m.boundingRect(QRect(), 0, name_).width(), m.height());
+ m.boundingRect(QRect(), 0, base_->name()).width(), m.height());
const QSizeF label_size(
text_size.width() + LabelPadding.width() * 2,
ceilf((text_size.height() + LabelPadding.height() * 2) / 2) * 2);
@@ -166,32 +151,31 @@ QRectF Trace::label_rect(const QRectF &rect) const
label_size.height());
}
-QRectF Trace::hit_box_rect(const ViewItemPaintParams &pp) const
+void Trace::paint_back(QPainter &p, ViewItemPaintParams &pp)
{
- const float h = QFontMetrics(QApplication::font()).height();
- return QRectF(pp.left(), get_visual_y() - h / 2.0f,
- pp.width(), h);
-}
+ const View *view = owner_->view();
+ assert(view);
-void Trace::paint_back(QPainter &p, const ViewItemPaintParams &pp)
-{
- p.setPen(QPen(Qt::NoPen));
- p.setBrush(bgcolour_);
-
- const std::pair extents = v_extents();
+ if (view->coloured_bg())
+ p.setBrush(base_->bgcolour());
+ else
+ p.setBrush(pp.next_bg_colour_state() ? BrightGrayBGColour : DarkGrayBGColour);
- const int x = 0;
- const int y = get_visual_y() + extents.first;
- const int w = pp.right() - pp.left();
- const int h = extents.second - extents.first;
+ p.setPen(QPen(Qt::NoPen));
- p.drawRect(x, y, w, h);
+ const pair extents = v_extents();
+ p.drawRect(pp.left(), get_visual_y() + extents.first,
+ pp.width(), extents.second - extents.first);
}
-void Trace::paint_axis(QPainter &p, const ViewItemPaintParams &pp, int y)
+void Trace::paint_axis(QPainter &p, ViewItemPaintParams &pp, int y)
{
+ p.setRenderHint(QPainter::Antialiasing, false);
+
p.setPen(AxisPen);
- p.drawLine(QPointF(pp.left(), y + 0.5f), QPointF(pp.right(), y + 0.5f));
+ p.drawLine(QPointF(pp.left(), y), QPointF(pp.right(), y));
+
+ p.setRenderHint(QPainter::Antialiasing, true);
}
void Trace::add_colour_option(QWidget *parent, QFormLayout *form)
@@ -201,9 +185,9 @@ void Trace::add_colour_option(QWidget *parent, QFormLayout *form)
ColourButton *const colour_button = new ColourButton(
TracePalette::Rows, TracePalette::Cols, parent);
colour_button->set_palette(TracePalette::Colours);
- colour_button->set_colour(colour_);
+ colour_button->set_colour(base_->colour());
connect(colour_button, SIGNAL(selected(const QColor&)),
- this, SLOT(on_colour_changed(const QColor&)));
+ this, SLOT(on_colouredit_changed(const QColor&)));
form->addRow(tr("Colour"), colour_button);
}
@@ -213,10 +197,14 @@ void Trace::create_popup_form()
// Clear the layout
// Transfer the layout and the child widgets to a temporary widget
- // which then goes out of scope destroying the layout and all the child
- // widgets.
- if (popup_form_)
- QWidget().setLayout(popup_form_);
+ // which we delete after the event was handled. This way, the layout
+ // and all widgets contained therein are deleted after the event was
+ // handled, leaving the parent popup_ time to handle the change.
+ if (popup_form_) {
+ QWidget *suicidal = new QWidget();
+ suicidal->setLayout(popup_form_);
+ suicidal->deleteLater();
+ }
// Repopulate the popup
popup_form_ = new QFormLayout(popup_);
@@ -227,35 +215,62 @@ void Trace::create_popup_form()
void Trace::populate_popup_form(QWidget *parent, QFormLayout *form)
{
QLineEdit *const name_edit = new QLineEdit(parent);
- name_edit->setText(name_);
+ name_edit->setText(base_->name());
connect(name_edit, SIGNAL(textChanged(const QString&)),
- this, SLOT(on_text_changed(const QString&)));
+ this, SLOT(on_nameedit_changed(const QString&)));
form->addRow(tr("Name"), name_edit);
add_colour_option(parent, form);
}
-void Trace::on_popup_closed()
+void Trace::set_name(QString name)
{
- popup_ = nullptr;
- popup_form_ = nullptr;
+ base_->set_name(name);
}
-void Trace::on_text_changed(const QString &text)
+void Trace::set_colour(QColor colour)
{
- set_name(text);
+ base_->set_colour(colour);
+}
- if (owner_)
+void Trace::on_name_changed(const QString &text)
+{
+ /* This event handler is called by SignalBase when the name was changed there */
+ (void)text;
+
+ if (owner_) {
owner_->extents_changed(true, false);
+ owner_->row_item_appearance_changed(true, false);
+ }
}
void Trace::on_colour_changed(const QColor &colour)
{
- set_colour(colour);
+ /* This event handler is called by SignalBase when the colour was changed there */
+ (void)colour;
if (owner_)
owner_->row_item_appearance_changed(true, true);
}
-} // namespace view
+void Trace::on_popup_closed()
+{
+ popup_ = nullptr;
+ popup_form_ = nullptr;
+}
+
+void Trace::on_nameedit_changed(const QString &name)
+{
+ /* This event handler notifies SignalBase that the name changed */
+ set_name(name);
+}
+
+void Trace::on_colouredit_changed(const QColor &colour)
+{
+ /* This event handler notifies SignalBase that the colour changed */
+ set_colour(colour);
+}
+
+} // namespace TraceView
+} // namespace views
} // namespace pv