QPaintEvent in MainWindow
-
I have the QMainWindow widget and two QLabel widgets inside it to display images and one QCheckBox widgte. In those two labels I display two images and after I check the Checkbox option I want that in one of those images some figures would be drawn on top of the displayed image. After I uncheck the checbox I want that the image returns as it was before painting all the stuff. I know that QPaintEvent can be used here but I don't want it to be trigged so often as some other activities are done in QMainWindow which would cause to trigger this event too often. So far I have this:
void MainWindow::paintEvent(QPaintEvent *e) { if (ui->checkBox->isChecked()) { QPainter painter(&image); painter.drawLine(10, 10, 150, 150); ui->label->setPixmap(QPixmap::fromImage(image)); } else { // back to original image } }
Is there a way to control paintEvent that it would be triggered only when I check the checkbox? Because now it will be triggered many times and else part will be done all the time when checkbox is not checked.
-
Hi,
Why not connect a slot to the toggled signal for the QCheckBox and change the image there ?
-
I only need that some figures are drawn on top of the existing image with QPainter whenever the checkbox is checked and when I uncheck it returns to the original image. I read that QPainter can be used only inside painterEvent, therefore, I don't see how toggled signal would help me unless you could clarify it :)
-
@Donn
If the figures are drawn over the label, why are you drawing them in the main window's paint event? And, no you can't or rather shouldn't depend on when the paint events are scheduled. You can get multiple spontaneous (such that come from the underlying OS) paint events when a widget is shown/overlapped/revealed. The simplest way would be to derive from theQLabel
or evenQWidget
and draw your pixmap manually, then when there's a paint event just do the painting. The toggling of the checkbox would simply set an internal flag for your custom widget. Something like this (untested, but with a bit of tweaking should work fine):class MyLabelWithOverlay : public QLabel { public: MyLabelWithOverlay (QWidget * parent = NULL) : QLabel(parent), overlayEnabled(false) { } public slots: void enableOverlay(bool enabled) { overlayEnabled = enabled; } protected: virtual void paintEvent(QPaintEvent * event) { QPixmap pix = pixmap(); qreal ratio = qMin(static_cast<qreal>(width()) / pix.width(), static_cast<qreal>(height()) / pix.height()); QTransform transform; transform.scale(ratio, ratio); QRect sourceRect = transform.mapRect(event->rect()); QPainter painter(this); painter.drawPixmap(event->rect(), pix, sourceRect); if (overlayEnabled) { // Draw your lines here } } private: bool overlayEnabled; }
You connect the
QCheckBox
'stoggled(bool)
signal to theenableOverlay(bool)
slot and that's mostly it.EDIT: I've added the proper casting where the ratio is calculated.
Kind regards.
-
I have created a header file
#ifndef MYLABELWITHOVERLAY_H #define MYLABELWITHOVERLAY_H #include "QtGui" #include "QtCore" #include "QLabel" class MyLabelWithOverlay : public QLabel { Q_OBJECT public: MyLabelWithOverlay (QWidget * parent = NULL); public slots: void enableOverlay(bool enabled); protected: virtual void paintEvent(QPaintEvent * event); private: bool overlayEnabled; }; #endif // MYLABELWITHOVERLAY_H
and a source file
#include "mylabelwithoverlay.h" MyLabelWithOverlay::MyLabelWithOverlay (QWidget * parent): QLabel(parent), overlayEnabled(false) { } void MyLabelWithOverlay::enableOverlay(bool enabled) { overlayEnabled = enabled; } void MyLabelWithOverlay::paintEvent(QPaintEvent * event) { QPixmap pix = QPixmap(); qreal ratio = qMin(static_cast<qreal>(width()) / pix.width(), static_cast<qreal>(height()) / pix.height()); QTransform transform; transform.scale(ratio, ratio); QRect sourceRect = transform.mapRect(event->rect()); QPainter painter(this); painter.drawPixmap(event->rect(), pix, sourceRect); if (overlayEnabled) { // Draw your lines here } }
After that I promote QLabel widget in Designer to mylabelwithoverlay and try to run. I get the following errors in ui_mainwindow.h:
C2143: syntax error: missing ';' before '*' C4430: missing type specifier - int assumed.
Where could be the problem? I cannot edit ui_mainwindow.h file since it is auto generated. If I don't promote the widget the code compiles successfully after including mylabelwithoverlay.h file.
EDIT: I have spelled the class name in lowercase only instead of MyLabelWithOverlay. Now it compiles fine but I cannot make any changes to this label.
-
@Donn
Hello,
Firstly, thisQPixmap pix = QPixmap();
line is wrong. If you look at my example you'll see that the idea is not to create an empty pixmap but to get the image from the parent class' property. Secondly, I don't understand what changes you want to do with this label. You put your code that draws the lines, or whatsoever you wanted to draw, inside the if block (as indicated by the comment) and that's it. Did I misunderstand the problem in the first place?Kind regards.
-
For now I have promoted the QLabel widget label to MyLabelWithOverlay and I assume I can work with it as with a standard QLabel widget being able to change a text of this label, for example. Therefore, I tried to change the text by label->setText("lalal") but it doesn't work, no changes appear. Did I misunderstand something?
P.S. So far I'm not trying to draw some lines but trying to understand how it works with this new label and I think I will do the rest when I get it. Of course, my goal is what you have described.
-
@Donn
Ah, I see. Well since this label of sorts is providing its own implementation of the painting, the parent class' painting method will not be invoked automatically. This means that if you want to see the text you set withlabel->setText("lalal")
you'll have to paint it yourself in thevoid MyLabelWithOverlay::paintEvent(QPaintEvent * event)
method. Does this clear it a bit? You could try setting a pixmap for the label (this should be displayed, although how well it's painted I can't vouch as I've written the example directly in the forum).Kind regards.
-
@Donn
It inherits all of the methods provided byQLabel
, but it overrides the QLabel::paintEvent method, so no painting is done fromQLabel::paintEvent
and consequently you don't get the text painted (which is whatQLabel::paintEvent
actually does). -
So QLabel::paintEvent is actually responsible for painting a simple text and if I override it doesn't provide the functionality for displaying text anymore and I need to do it myself in the body of overridden function, right? Can you provide some code how it could be in the body because I don't have any idea.
-
@Donn said:
So QLabel::paintEvent is actually responsible for painting a simple text and if I override it doesn't provide the functionality for displaying text anymore and I need to do it myself in the body of overridden function, right?
Correct.
Can you provide some code how it could be in the body because I don't have any idea.
void MyLabelWithOverlay::paintEvent(QPaintEvent * event) { // This code draws the pixmap that was given with setPixmap(), and hopefully it does that correctly QPixmap pix = pixmap(); qreal ratio = qMin(static_cast<qreal>(width()) / pix.width(), static_cast<qreal>(height()) / pix.height()); QTransform transform; transform.scale(ratio, ratio); QRect sourceRect = transform.mapRect(event->rect()); QPainter painter(this); painter.drawPixmap(event->rect(), pix, sourceRect); //< Pixmap was drawn if (overlayEnabled) { //< Here you can draw on top of that pixmap the lines (that's why it was left unfilled) } // Here you can draw the text (on top of the pixmap and the lines QRect widgetRect = rect(); //< This is the widget's rectangle QString labelText = text(); //< This is the text that was set // This following line draws the text centered horizontally and bound by the provided rectangle painter.drawText(widgetRect, labelText, Qt::AlignHCenter); }
-
void MyLabelWithOverlay::paintEvent(QPaintEvent * event) { const QPixmap pix = *pixmap(); qreal ratio = qMin(static_cast<qreal>(width()) / pix.width(), static_cast<qreal>(height()) / pix.height()); QTransform transform; transform.scale(ratio, ratio); QRect sourceRect = transform.mapRect(event->rect()); QPainter painter(this); painter.drawPixmap(event->rect(), pix, sourceRect); if (overlayEnabled) { // Draw your lines here } // Here you can draw the text (on top of the pixmap and the lines QRectF widgetRect = rect(); //< This is the widget's rectangle QString labelText = text(); //< This is the text that was set // This following line draws the text centered horizontally and bound by the provided rectangle //painter.drawText(widgetRect, labelText, Qt::AlignHCenter); }
My app crashes after this saying that
QWidget::repaint: Recursive repaint detected
If I uncomment the last line I get the following:
C2664: 'void QPainter::drawText(const QRectF &,const QString &,const QTextOption &)': cannot convert argument 1 from 'QRectF' to 'int'
-
const QPixmap pix = *pixmap();
Since you're not checking if the pointer returned is NULL, have you set a pixmap for the label with
QLabel::setPixmap
?
If you haven't this would cause a segfault, because dereferencing a NULL pointer isn't allowed.Recursive repaint detected
This shouldn't be normally happening. Are you calling
repaint()
manually, or is there an explicit call topaintEvent()
somewhere?If I uncomment the last line I get the following: ... cannot convert argument 1 from 'QRectF' to 'int'
Have you included the header for
QRectF
? -
@kshegunov Wouldn't it be easier to first call QLabel::paintEvent(...) to get back the inherited functionality and then draw custom stuff?
-
@kshegunov said:
Since you're not checking if the pointer returned is NULL, have you set a pixmap for the label with
QLabel::setPixmap
?
If you haven't this would cause a segfault, because dereferencing a NULL pointer isn't allowed.Could you please provide a solution for this because I really don't get it how it is done since it is a big mess for me and trying to get it step by step. I really appreatiate your help!
This shouldn't be normally happening. Are you calling
repaint()
manually, or is there an explicit call topaintEvent()
somewhere?I do not call any repaint() or paintEvent() explicitly I think. I just simply created a header file and a source file as mentioned above and have included header file to other header file and have promoted one QLabel widget to MyLabelWithOverlay.
Have you included the header for
QRectF
?I have included the header but it is still the same problem.
-
@jsulm
Absolutely! Although, I was half way down writing the code when I realized I had aQLabel
instead ofQWidget
(which would be usually the case) and was reluctant to rewrite it. :)Could you please provide a solution for this because I really don't get it how it is done since it is a big mess for me and trying to get it step by step.
Solution for what you mean? How to check whether a pointer is null? Simply check with if:
QPixmap * pixmapPointer = pixmap(); if (!pixmapPointer) return; QPixmap & pix = *pixmapPointer; //< You have to make sure pixmapPointer points to a valid object before using the deref operator (the *)
Also something I noticed in your code is that you're passing
const QPixmap *
toQPainter::drawPixmap
, which acceptsconst QPixmap &
, you should fix that (look the snippet how conversion between the two is done).I have included the header but it is still the same problem.
Usually this happens when the header is not present. Try passing the object directly:
painter.drawText(QRectF(rect()), labelText, Qt::AlignHCenter);
-
Everything mentioned above compiles except the last problem:
Usually this happens when the header is not present. Try passing the object directly:
painter.drawText(QRectF(rect()), labelText, Qt::AlignHCenter);
I still get the same error message. Instead I tried
painter.drawText(20, 20, labelText);
which compiles but I don't get any text displayed in the promoted MyLabelWithOverlay label.
EDIT:
Maybe I'm doing something wrong in other place. If you can, please check it because I have no idea what could be wrong. Perhaps somewhere I don't completely get how things work...
mainwindow.h:
#ifndef MAINWINDOW_H #define MAINWINDOW_H #include <QMainWindow> namespace Ui { class MainWindow; } class MainWindow : public QMainWindow { Q_OBJECT public: explicit MainWindow(QWidget *parent = 0); ~MainWindow(); private: Ui::MainWindow *ui; }; #endif // MAINWINDOW_H
MainWindow.cpp:
#include "mainwindow.h" #include "ui_mainwindow.h" #include "mylabelwithoverlay.h" MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent), ui(new Ui::MainWindow) { ui->setupUi(this); ui->label_5->setText("hello world"); // label_5 is promoted to MyLabelWithOverlay } MainWindow::~MainWindow() { delete ui; }
mylabelwithoverlay.h:
#ifndef MYLABELWITHOVERLAY_H #define MYLABELWITHOVERLAY_H #include "QtGui" #include "QtCore" #include "QLabel" class MyLabelWithOverlay : public QLabel { Q_OBJECT public: MyLabelWithOverlay (QWidget * parent = NULL); public slots: void enableOverlay(bool enabled); protected: virtual void paintEvent(QPaintEvent * event); private: bool overlayEnabled; }; #endif // MYLABELWITHOVERLAY_H
MyLabelWithOverlay.cpp:
#include "mylabelwithoverlay.h" #include <QRect> #include <QRectF> MyLabelWithOverlay::MyLabelWithOverlay (QWidget * parent): QLabel(parent), overlayEnabled(false) { } void MyLabelWithOverlay::enableOverlay(bool enabled) { overlayEnabled = enabled; } void MyLabelWithOverlay::paintEvent(QPaintEvent * event) { const QPixmap * pixmapPointer = pixmap(); if (!pixmapPointer) return; const QPixmap & pix = *pixmapPointer; qreal ratio = qMin(static_cast<qreal>(width()) / pix.width(), static_cast<qreal>(height()) / pix.height()); QTransform transform; transform.scale(ratio, ratio); QRect sourceRect = transform.mapRect(event->rect()); QPainter painter(this); painter.drawPixmap(event->rect(), pix, sourceRect); if (overlayEnabled) { // Draw your lines here } // Here you can draw the text (on top of the pixmap and the lines QRect widgetRect = rect(); //< This is the widget's rectangle QString labelText = text(); //< This is the text that was set // This following line draws the text centered horizontally and bound by the provided rectangle //painter.drawText(QRectF(rect()), labelText, Qt::AlignHCenter); painter.drawText(20, 20, labelText); }
-
@Donn
You're not setting a pixmap for the label, so when the paint events gets executedpixmapPointer
will be NULL and the function will return. If you wish to support both/either text and pixmap you have to enclose each in its own if block.void MyLabelWithOverlay::paintEvent(QPaintEvent * event) { QPainter painter(this); const QPixmap * pixmapPointer = pixmap(); if (pixmapPointer) { const QPixmap & pix = *pixmapPointer; qreal ratio = qMin(static_cast<qreal>(width()) / pix.width(), static_cast<qreal>(height()) / pix.height()); QTransform transform; transform.scale(ratio, ratio); QRect sourceRect = transform.mapRect(event->rect()); painter.drawPixmap(event->rect(), pix, sourceRect); } if (overlayEnabled) { // Draw your lines here } QString labelText = text(); if (!labelText.empty()) { painter.drawText(0, 0, labelText); } }
-
I think I got it! I have included to the painterEvent the following:
QPixmap pm(200, 200); pm.fill(Qt::white); setPixmap(pm);
Now I can set the text from other class like:
ui->label->setText("lala");
However, I think I shouldn't set Pixmap inside paintEvent - better to do it in the other class so this type of label could be also used for other labels, right?
EDIT:
I liked the idea of @jsulm. Could you give me some hints how to achieve it? How to get the painting functionality from a standard QLabel and then add some additional my own painting?