Mouse Pointer and Label Background Coloring in Qt Application with Multiple Labels

JonB

@Collaxd said in Mouse Pointer and Label Background Coloring in Qt Application with Multiple Labels:

When I click on a label, the mouse pointer becomes stuck on that label, and I am unable to click on other nearby labels.

Maybe because you do not call the base class implementation for your mouse...Event() overrides?
Separately I don't know whether qDebug() output on every mouse move makes it sluggish.

Collaxd

@JonB

protected:
    void mousePressEvent(QMouseEvent *event) override;
    void mouseMoveEvent(QMouseEvent *event) override;
    void enterEvent(QEnterEvent *event) override;
    void leaveEvent(QEvent *event) override;

thanks for your answer yes and it does everything just using override, any suggestions?

SGaist

Hi,

@JonB was not talking about the declaration of the functions but calling the base class implementation in your reimplementation.

On an unrelated note, your use of parent to do the connection in the constructor is bad. That connection should happen in the parent. A child widget should know nothing about its parent.

Collaxd

@SGaist
thanks for your answer. I understand better now, I'm learning and I don't know how the best declaration was, so I chose to do it inside the child, but now I've implemented it in the creation of the labels
but the problem persists

Sorry but I still can't understand the implementation part, I'm a little new to cpp sorry


MainWindow::MainWindow(QWidget *parent)
    : QMainWindow(parent)
    , ui(new Ui::MainWindow)
    , m_hands (readHands())
{
    ui->setupUi(this);

    int row=0;
    int col=0;
    foreach (QString hand, m_hands) {
        m_label = new HandLabel(hand, this);
        connect(this, SIGNAL(bgColorChanged(QColor)), m_label, SLOT(onbgColorChanged(QColor)));

        ui->handsContainer->addWidget(m_label, row, col);
        
        if (col < 12) col++;
        else {
            col=0;
            row++;
        }
    }

    emit bgColorChanged(Qt::yellow);
}```

SGaist

@Collaxd as I said, it was an unrelated note.

For your issue: do you know what "calling the base class implementation" means ? That's key to answer @JonB's question.

Collaxd

@SGaist said in Mouse Pointer and Label Background Coloring in Qt Application with Multiple Labels:

For your issue: do you know what "calling the base class implementation" means ? That's key to answer @JonB's question.

no, i even asked gpt chat and tried to learn but i don't understand what jon is getting at, could you simplify the question so i can improve my example?

Christian Ehrlicher

@Collaxd said in Mouse Pointer and Label Background Coloring in Qt Application with Multiple Labels:

no, i even asked gpt chat

I really wonder where such stuff will end... very staggering.

When you don't want to read a good c++ book then you can ask google for 'calling the base class implementation'.

JoeCFD

@Collaxd

void HandLabel::mousePressEvent(QMouseEvent *event)
{
    Q_UNUSED(event);

    QPalette pal = palette();
    QColor selectedColor = pal.color(QPalette::Window);

    if (m_bgColor == selectedColor) {
        pal.setColor(QPalette::Window, Qt::transparent);
    } else {
        pal.setColor(QPalette::Window, m_bgColor);
    }
    setPalette(pal);

   /* something like this */
   QLabel::mousePressEvent( event );
}

SGaist

@Collaxd you better go with a good old fashioned cpp tutorial about that very subject.

Collaxd

@Christian-Ehrlicher
Thank you, friends. I really didn't mean to offend when I mentioned the chat. In fact, I'm just a veterinarian enthusiast in programming trying to learn some things. I'll do more research.

JonB

@Collaxd
No offence taken.

In, say, your HandLabel::enterEvent() overridden method you call

QLabel::enterEvent(event); // chama o enterEvent da classe base

Same for leaveEvent(). It allows whatever the class you derive from to see these events and act on them, as well as your own code.

I would guess you need to do the same for your HandLabel::mousePressEvent(). I would let it call QLabel::mousePressEvent(event), as well as your code. Maybe also for mouseMoveEvent(). Does that help with your

When I click on a label, the mouse pointer becomes stuck on that label, and I am unable to click on other nearby labels.

?

Collaxd

@JonB
Perfect, thanks for your understanding and help, but even when calling the base event, the mouse is still 'stuck' on the initial click label, I can do something just with enter, it also serves my purpose, but it doesn't work in the same way

Here's an example of how my code is and an app I want to 'imitate' with it working

#include "HandLabel.h"

#include <qapplication.h>

HandLabel::HandLabel(const QString &text, QWidget *parent, Qt::WindowFlags f)
    : QLabel(text, parent, f)
{
    setAlignment(Qt::AlignCenter);
    setFrameStyle(QFrame::Panel | QFrame::Sunken);
    setAutoFillBackground(true);
}


void HandLabel::onbgColorChanged(QColor newColor)
{
    setBgColor(newColor);
}

void HandLabel::mousePressEvent(QMouseEvent *event)
{
    QLabel::mousePressEvent(event); // chama o mousePressEvent da classe base

    if (event->buttons() & Qt::LeftButton)
        updateBackground();

}

void HandLabel::mouseMoveEvent(QMouseEvent *event)
{
    QLabel::mouseMoveEvent(event);

    qDebug() << event;
//    updateBackground();

}

void HandLabel::enterEvent(QEnterEvent *event)
{
    QLabel::enterEvent(event); // chama o enterEvent da classe base
    updateBackground();
    setMouseTracking(true);
}

void HandLabel::leaveEvent(QEvent *event)
{
    QLabel::leaveEvent(event); // chama o leaveEvent da classe base
    // emite um evento mouseReleaseEvent
    setMouseTracking(false);
}

void HandLabel::updateBackground()
{
    QPalette pal = palette();
    QColor selectedColor = pal.color(QPalette::Window);
    if (m_bgColor == selectedColor)
        pal.setColor(QPalette::Window, Qt::transparent);
    else
        pal.setColor(QPalette::Window, m_bgColor);

    setPalette(pal);
}


QColor HandLabel::bgColor() const
{
    return m_bgColor;
}

void HandLabel::setBgColor(const QColor &newBgColor)
{
    m_bgColor = newBgColor;
}

alt text

mpergand

@Collaxd said in Mouse Pointer and Label Background Coloring in Qt Application with Multiple Labels:

he mouse is still 'stuck' on the initial click label,

I think you should use dragEnterEvent & dragMoveEvent instead of enterEvent, if you want to select muti objects while mouse press.

Collaxd

@mpergand

void HandLabel::dragEnterEvent(QDragEnterEvent *event)
{
    qDebug() << event;
    QLabel::dragEnterEvent(event);
}

void HandLabel::dragMoveEvent(QDragMoveEvent *event)
{
    qDebug() << event;
    QLabel::dragMoveEvent(event);
}

Thanks for your help, but it doesn't appear in the debug when I add the two functions :(

I managed to do something similar in React + JS but I put the lstenner in the parent layout which is not possible here since the parent layout is a grid and has no mouse properties

mpergand

@Collaxd
Seems to work:

class Label : public QLabel
{
    public:
    Label(const QString& text, QWidget* parent=nullptr) : QLabel(text,parent)
    {
    setFrameStyle(QFrame::Panel | QFrame::Sunken);
    setAcceptDrops(true);
    }

    void mousePressEvent(QMouseEvent *ev) override
    {
         qDebug()<<Q_FUNC_INFO;
         QLabel::mousePressEvent(ev);
    }

    void mouseMoveEvent(QMouseEvent *ev) override
    {
         qDebug()<<Q_FUNC_INFO;
        QDrag *drag = new QDrag(this);
        QMimeData *mimeData = new QMimeData;

        mimeData->setText("");
        drag->setMimeData(mimeData);

        Qt::DropAction dropAction = drag->exec(Qt::MoveAction);
        if(  dropAction == Qt::IgnoreAction )
        {
            qDebug()<<"end drag";
            drag->deleteLater();
        }
    }


    void dragEnterEvent(QDragEnterEvent* ev ) override
    {
        qDebug()<<Q_FUNC_INFO;
         ev->acceptProposedAction();
    }

    void dragMoveEvent(QDragMoveEvent* ev ) override
    {
        qDebug()<<Q_FUNC_INFO<<ev->source()<<this;

    }
};

Each Label receives dragMoveEvent when the mouse is over.
Looks promising, have a look at the doc about drag&drop:
https://doc.qt.io/qt-5/dnd.html

Collaxd

@mpergand
I would like to thank you very much for your solution, it was a success! I've been searching for days and couldn't find anything similar, which generated a lot of learning and made me realize that I still have a lot to study!

For future readers, here is the solution:

void HandLabel::mouseMoveEvent(QMouseEvent *event)
{
    Q_UNUSED(event);

    QDrag *drag = new QDrag(this);

    QMimeData *mimeData = new QMimeData;
    drag->setMimeData(mimeData);

    Qt::DropAction dropAction = drag->exec(Qt::MoveAction);
    if(  dropAction == Qt::IgnoreAction ) {
        drag->deleteLater();
    }
}

void HandLabel::dragEnterEvent(QDragEnterEvent *event)
{
    Q_UNUSED(event);
    updateColorScheme();
}

Despite the mouse having a 'block' symbol, the colors are working perfectly!

alt text

mpergand

@Collaxd said in Mouse Pointer and Label Background Coloring in Qt Application with Multiple Labels:

void HandLabel::dragEnterEvent(QDragEnterEvent *event)
{
Q_UNUSED(event);
updateColorScheme();
}

Oh, miss that drag enter is call for each label !
Even easier that I imagined.

Despite the mouse having a 'block' symbol, the colors are working perfectly!

You need to call ev->acceptProposedAction();
for that symbol to not appear.