QWidget subclass drawing in wrong place

[guidupas]

Hello all!

I hava a QWidget subclassed and the painting is not going right. When I call setFluxo all the rects appear in the screen on position y 0

#include "fluxocaixawidget.h"
 
#include <QDebug>
 
fluxoCaixaWidget::fluxoCaixaWidget(QWidget *parent) :
    QWidget(parent)
{
    this->setMouseTracking(true);
    dentro = false;
    fluxo = NULL;
 
    this->update();
}
 
void fluxoCaixaWidget::paintEvent(QPaintEvent *)
{
    QPainter painter(this);
 
    linhaTempo.setRect(20, this->parentWidget()->height() / 2 - 20, this->parentWidget()->width()-65, 30);
 
    QPen caneta;
 
    QBrush pincel(Qt::white);
    painter.setBrush(pincel);
 
    painter.drawRect(parentWidget()->rect());
 
    if(this->dentro == true)
    {
        painter.drawText(QRect(20, 15, linhaTempo.width(), 25), "Linha do tempo", QTextOption::QTextOption(Qt::AlignCenter));
        caneta.setWidth(4);
    }
    else
    {
        caneta.setWidth(2);
    }
 
    painter.setPen(caneta);
 
    painter.drawRect(linhaTempo);
 
    if(fluxo != NULL)
    {
        for(int i = 0; i < fluxo->count(); i++)
        {
            if(fluxo->at(i) < 0)
            {
                caneta.setColor(Qt::red);
                caneta.setWidth(2);
 
                painter.setPen(caneta);
                painter.drawRect(fluxoRect.at(i));
            }
            else
            {
                if(fluxo->at(i) == 0)
                {
                    caneta.setColor(Qt::black);
                    caneta.setWidth(2);
 
                    painter.setPen(caneta);
                    painter.drawRect(fluxoRect.at(i));
                }
                else
                {
                    if(fluxo->at(i) > 0)
                    {
                        caneta.setColor(Qt::blue);
                        caneta.setWidth(2);
 
                        painter.setPen(caneta);
                        painter.drawRect(fluxoRect.at(i));
                    }
                }
            }
        }
    }
}
 
void fluxoCaixaWidget::mouseMoveEvent(QMouseEvent *)
{
    if(this->linhaTempo.contains(this->mapFromGlobal(QCursor::pos())))
    {
        dentro = true;
    }
    else
    {
        dentro = false;
    }
    this->update();
}
 
void fluxoCaixaWidget::setFluxo(QList<double> *f)
{
    fluxo = f;
 
    fluxoRect.clear();
 
    int espacamento = 0;
 
    if(fluxo->count() > 1)
    {
        espacamento = linhaTempo.width() / (fluxo->count() - 1);
    }
 
    for(int i = 0; i < fluxo->count(); i++)
    {
        if(i == 0)
        {
            fluxoRect.append(QRect(20, this->parentWidget()->height() / 2 - 20, 10, 50));
        }
        else
        {
            fluxoRect.append(QRect(20 + espacamento * i, this->parentWidget()->height() / 2 - 20, 10, 50));
        }
    }
 
    this->update();
}

#include "fluxocaixawidget.h"

#include <QDebug>

fluxoCaixaWidget::fluxoCaixaWidget(QWidget *parent) :
    QWidget(parent)
{
    this->setMouseTracking(true);
    dentro = false;
    fluxo = NULL;

    this->update();
}

void fluxoCaixaWidget::paintEvent(QPaintEvent *)
{
    QPainter painter(this);

    linhaTempo.setRect(20, this->parentWidget()->height() / 2 - 20, this->parentWidget()->width()-65, 30);

    QPen caneta;

    QBrush pincel(Qt::white);
    painter.setBrush(pincel);

    painter.drawRect(parentWidget()->rect());

    if(this->dentro == true)
    {
        painter.drawText(QRect(20, 15, linhaTempo.width(), 25), "Linha do tempo", QTextOption::QTextOption(Qt::AlignCenter));
        caneta.setWidth(4);
    }
    else
    {
        caneta.setWidth(2);
    }

    painter.setPen(caneta);

    painter.drawRect(linhaTempo);

    if(fluxo != NULL)
    {
        for(int i = 0; i < fluxo->count(); i++)
        {
            if(fluxo->at(i) < 0)
            {
                caneta.setColor(Qt::red);
                caneta.setWidth(2);

                painter.setPen(caneta);
                painter.drawRect(fluxoRect.at(i));
            }
            else
            {
                if(fluxo->at(i) == 0)
                {
                    caneta.setColor(Qt::black);
                    caneta.setWidth(2);

                    painter.setPen(caneta);
                    painter.drawRect(fluxoRect.at(i));
                }
                else
                {
                    if(fluxo->at(i) > 0)
                    {
                        caneta.setColor(Qt::blue);
                        caneta.setWidth(2);

                        painter.setPen(caneta);
                        painter.drawRect(fluxoRect.at(i));
                    }
                }
            }
        }
    }
}

void fluxoCaixaWidget::mouseMoveEvent(QMouseEvent *)
{
    if(this->linhaTempo.contains(this->mapFromGlobal(QCursor::pos())))
    {
        dentro = true;
    }
    else
    {
        dentro = false;
    }
    this->update();
}

void fluxoCaixaWidget::setFluxo(QList<double> *f)
{
    fluxo = f;

    fluxoRect.clear();

    int espacamento = 0;

    if(fluxo->count() > 1)
    {
        espacamento = linhaTempo.width() / (fluxo->count() - 1);
    }

    for(int i = 0; i < fluxo->count(); i++)
    {
        if(i == 0)
        {
            fluxoRect.append(QRect(20, this->parentWidget()->height() / 2 - 20, 10, 50));
        }
        else
        {
            fluxoRect.append(QRect(20 + espacamento * i, this->parentWidget()->height() / 2 - 20, 10, 50));
        }
    }

    this->update();
}

To copy to clipboard, switch view to plain text mode 

[anda_skoa]

Why are you accessing parentWidget()?

Aside from it not making any sense, what if there is no parent widget?

Cheers,
_

[guidupas]

Yes, you are right. But how can I have a size reference to create the QRect linhaTempo?. And the QRect linhaTempo must be resizable.

Thank you for the reply.

Cheers.

[anda_skoa]

Your own widget has a size as well, does it not?

Cheers,
_

[guidupas]

Yes, it has, but its does not resize when it has a fixed size.

Could you help me?

[d_stranz]

Yes, it has, but its does not resize when it has a fixed size.

Are you using layouts in the parent widget? If not, this is almost certainly the cause of your widget not resizing. If you put your widget inside a layout on the parent widget, the layout will automatically resize your widget when the parent resizes.

[guidupas]

Yes. My widget is inside a layout on the parent widget. But it does not resize

QGridLayout *gridLayoutInvestimento5 = new QGridLayout(boxInvestimento3);
 
    fluxoCaixaWidget *fluxoCaixa = new fluxoCaixaWidget(boxInvestimento3);
 
    boxInvestimento3->setLayout(gridLayoutInvestimento5);
    gridLayoutInvestimento5->addWidget(fluxoCaixa);

QGridLayout *gridLayoutInvestimento5 = new QGridLayout(boxInvestimento3);

    fluxoCaixaWidget *fluxoCaixa = new fluxoCaixaWidget(boxInvestimento3);

    boxInvestimento3->setLayout(gridLayoutInvestimento5);
    gridLayoutInvestimento5->addWidget(fluxoCaixa);

To copy to clipboard, switch view to plain text mode 

[d_stranz]

Sorry, I can't tell you what is wrong. You aren't showing us enough of your code to understand where your problem is. Six lines of code without any idea of where those lines are being used isn't enough information.

When I am faced with a problem like this, I create a demo project that simplifies the UI to the most basic level. If the problem still occurs in the demo project, then I figure out what is wrong and I fix it. It is very difficult to diagnose UI problems inside of a much larger project because usually too many other things are happening. Isolate the problem to the most basic parts and fix the problem.

[guidupas]

Sorry, its because its a large code. I will put below a larger code

mainwindow.cpp

QWidget *MainWindow::criaEstruturaNovoIvestimento()
{
    QWidget *tabInvestimento = new QWidget(this->tabWidgetInvestimento);
 
    QGridLayout *gridLayoutInvestimento2 = new QGridLayout(tabInvestimento);
    gridLayoutInvestimento2->setMargin(0);
    QToolBox *toolboxInvestimento = new QToolBox(tabInvestimento);
    gridLayoutInvestimento2->addWidget(toolboxInvestimento);
    QWidget *boxInvestimento = new QWidget(toolboxInvestimento);
    QWidget *boxInvestimento2 = new QWidget(toolboxInvestimento);
    QWidget *boxInvestimento3 = new QWidget(toolboxInvestimento);
 
    //SOME CODE
 
    //HERE IS THE PROBLEM
    //Tool Box 3
    QGridLayout *gridLayoutInvestimento5 = new QGridLayout(boxInvestimento3);
 
    fluxoCaixaWidget *fluxoCaixa = new fluxoCaixaWidget(boxInvestimento3);
 
    boxInvestimento3->setLayout(gridLayoutInvestimento5);
    gridLayoutInvestimento5->addWidget(fluxoCaixa);
    //MORE CODE BELOW
}

QWidget *MainWindow::criaEstruturaNovoIvestimento()
{
    QWidget *tabInvestimento = new QWidget(this->tabWidgetInvestimento);

    QGridLayout *gridLayoutInvestimento2 = new QGridLayout(tabInvestimento);
    gridLayoutInvestimento2->setMargin(0);
    QToolBox *toolboxInvestimento = new QToolBox(tabInvestimento);
    gridLayoutInvestimento2->addWidget(toolboxInvestimento);
    QWidget *boxInvestimento = new QWidget(toolboxInvestimento);
    QWidget *boxInvestimento2 = new QWidget(toolboxInvestimento);
    QWidget *boxInvestimento3 = new QWidget(toolboxInvestimento);

    //SOME CODE

    //HERE IS THE PROBLEM
    //Tool Box 3
    QGridLayout *gridLayoutInvestimento5 = new QGridLayout(boxInvestimento3);

    fluxoCaixaWidget *fluxoCaixa = new fluxoCaixaWidget(boxInvestimento3);

    boxInvestimento3->setLayout(gridLayoutInvestimento5);
    gridLayoutInvestimento5->addWidget(fluxoCaixa);
    //MORE CODE BELOW
}

To copy to clipboard, switch view to plain text mode 

fluxocaixawidget.cpp

#include "fluxocaixawidget.h"
 
#include <QDebug>
 
fluxoCaixaWidget::fluxoCaixaWidget(QWidget *parent) :
    QWidget(parent)
{
    linhaTempo.setRect(20, 100, 700, 30);
 
    this->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding);
 
 
 
    this->setMouseTracking(true);
 
    dentro = false;
    fluxo = NULL;
}
 
void fluxoCaixaWidget::paintEvent(QPaintEvent *)
{
    QPainter painter(this);
 
    //linhaTempo.setRect(20, this->parentWidget()->height() / 2 - 20, this->parentWidget()->width()-65, 30);
 
    //linhaTempo.setRect(20, this->parentWidget()->rect().height() / 2 - 20, this->parentWidget()->rect().width()-65, 30);
 
    QPen caneta;
 
    QBrush pincel(Qt::white);
    painter.setBrush(pincel);
 
    //painter.drawRect(parentWidget()->rect());
    painter.drawRect(0,0,800,600);
 
    if(this->dentro == true)
    {
        painter.drawText(QRect(20, 15, linhaTempo.width(), 25), "Linha do tempo", QTextOption::QTextOption(Qt::AlignCenter));
        caneta.setWidth(3);
    }
    else
    {
        caneta.setWidth(2);
    }
 
    painter.setPen(caneta);
 
    painter.drawRect(linhaTempo);
 
    if(fluxo != NULL)
    {
        for(int i = 0; i < fluxo->count(); i++)
        {
            if(fluxo->at(i) < 0)
            {
                caneta.setColor(Qt::red);
            }
            else
            {
                if(fluxo->at(i) == 0)
                {
                    caneta.setColor(Qt::black);
                }
                else
                {
                    if(fluxo->at(i) > 0)
                    {
                        caneta.setColor(Qt::blue);
                    }
                }
            }
 
            if(fluxoDentro.at(i) == true)
            {
                QString valor = QString::number(fluxo->at(i));
                painter.drawText(QRect(20, 15, linhaTempo.width(), 25), valor, QTextOption::QTextOption(Qt::AlignCenter));
                caneta.setWidth(3);
            }
            else
            {
                caneta.setWidth(2);
            }
            painter.setPen(caneta);
            painter.drawRect(fluxoRect.at(i));
        }
    }
}
 
void fluxoCaixaWidget::mouseMoveEvent(QMouseEvent *)
{
    if(linhaTempo.contains(this->mapFromGlobal(QCursor::pos())))
    {
        dentro = true;
    }
    else
    {
        dentro = false;
    }
 
    for(int i = 0; i < fluxoRect.count(); i++)
    {
        if(fluxoRect.at(i).contains(this->mapFromGlobal(QCursor::pos())))
        {
            fluxoDentro.insert(i, true);
            dentro = false;
        }
        else
        {
            fluxoDentro.insert(i, false);
        }
    }
 
    this->update();
}
 
void fluxoCaixaWidget::setFluxo(QList<double> *f)
{
    fluxo = f;
 
    fluxoRect.clear();
    fluxoDentro.clear();
 
    int espacamento = 0;
 
    if(fluxo->count() > 1)
    {
        espacamento = linhaTempo.width() / (fluxo->count() - 1);
    }
 
    for(int i = 0; i < fluxo->count(); i++)
    {
        if(i == 0)
        {
            //fluxoRect.append(QRect(20, this->parentWidget()->height() / 2 - 20, 10, 50));
            fluxoRect.append(QRect(20, linhaTempo.y(), 10, 50));
            fluxoDentro.append(false);
        }
        else
        {
            //fluxoRect.append(QRect(20 + espacamento * i, this->parentWidget()->height() / 2 - 20, 10, 50));
            fluxoRect.append(QRect(20 + espacamento * i, linhaTempo.y(), 10, 50));
            fluxoDentro.append(false);
        }
    }
 
    this->update();
}

#include "fluxocaixawidget.h"

#include <QDebug>

fluxoCaixaWidget::fluxoCaixaWidget(QWidget *parent) :
    QWidget(parent)
{
    linhaTempo.setRect(20, 100, 700, 30);

    this->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding);



    this->setMouseTracking(true);

    dentro = false;
    fluxo = NULL;
}

void fluxoCaixaWidget::paintEvent(QPaintEvent *)
{
    QPainter painter(this);

    //linhaTempo.setRect(20, this->parentWidget()->height() / 2 - 20, this->parentWidget()->width()-65, 30);

    //linhaTempo.setRect(20, this->parentWidget()->rect().height() / 2 - 20, this->parentWidget()->rect().width()-65, 30);

    QPen caneta;

    QBrush pincel(Qt::white);
    painter.setBrush(pincel);

    //painter.drawRect(parentWidget()->rect());
    painter.drawRect(0,0,800,600);

    if(this->dentro == true)
    {
        painter.drawText(QRect(20, 15, linhaTempo.width(), 25), "Linha do tempo", QTextOption::QTextOption(Qt::AlignCenter));
        caneta.setWidth(3);
    }
    else
    {
        caneta.setWidth(2);
    }

    painter.setPen(caneta);

    painter.drawRect(linhaTempo);

    if(fluxo != NULL)
    {
        for(int i = 0; i < fluxo->count(); i++)
        {
            if(fluxo->at(i) < 0)
            {
                caneta.setColor(Qt::red);
            }
            else
            {
                if(fluxo->at(i) == 0)
                {
                    caneta.setColor(Qt::black);
                }
                else
                {
                    if(fluxo->at(i) > 0)
                    {
                        caneta.setColor(Qt::blue);
                    }
                }
            }

            if(fluxoDentro.at(i) == true)
            {
                QString valor = QString::number(fluxo->at(i));
                painter.drawText(QRect(20, 15, linhaTempo.width(), 25), valor, QTextOption::QTextOption(Qt::AlignCenter));
                caneta.setWidth(3);
            }
            else
            {
                caneta.setWidth(2);
            }
            painter.setPen(caneta);
            painter.drawRect(fluxoRect.at(i));
        }
    }
}

void fluxoCaixaWidget::mouseMoveEvent(QMouseEvent *)
{
    if(linhaTempo.contains(this->mapFromGlobal(QCursor::pos())))
    {
        dentro = true;
    }
    else
    {
        dentro = false;
    }

    for(int i = 0; i < fluxoRect.count(); i++)
    {
        if(fluxoRect.at(i).contains(this->mapFromGlobal(QCursor::pos())))
        {
            fluxoDentro.insert(i, true);
            dentro = false;
        }
        else
        {
            fluxoDentro.insert(i, false);
        }
    }

    this->update();
}

void fluxoCaixaWidget::setFluxo(QList<double> *f)
{
    fluxo = f;

    fluxoRect.clear();
    fluxoDentro.clear();

    int espacamento = 0;

    if(fluxo->count() > 1)
    {
        espacamento = linhaTempo.width() / (fluxo->count() - 1);
    }

    for(int i = 0; i < fluxo->count(); i++)
    {
        if(i == 0)
        {
            //fluxoRect.append(QRect(20, this->parentWidget()->height() / 2 - 20, 10, 50));
            fluxoRect.append(QRect(20, linhaTempo.y(), 10, 50));
            fluxoDentro.append(false);
        }
        else
        {
            //fluxoRect.append(QRect(20 + espacamento * i, this->parentWidget()->height() / 2 - 20, 10, 50));
            fluxoRect.append(QRect(20 + espacamento * i, linhaTempo.y(), 10, 50));
            fluxoDentro.append(false);
        }
    }

    this->update();
}

To copy to clipboard, switch view to plain text mode 

[guidupas]

when I do that, the QPushButton resizes but not my widget (fluxocaixawidget)

//Tool Box 3
    QGridLayout *gridLayoutInvestimento5 = new QGridLayout(boxInvestimento3);
 
    fluxoCaixaWidget *fluxoCaixa = new fluxoCaixaWidget(boxInvestimento3);
 
    QPushButton *button = new QPushButton("Teste", boxInvestimento3);
    QVBoxLayout *vertical = new QVBoxLayout(boxInvestimento3);
    vertical->addWidget(button);
    vertical->addWidget(fluxoCaixa);
 
    //gridLayoutInvestimento5->addWidget(fluxoCaixa);
 
    gridLayoutInvestimento5->addLayout(vertical,0,0);
    boxInvestimento3->setLayout(gridLayoutInvestimento5);
 
    //------------------------------------------------------------------------

//Tool Box 3
    QGridLayout *gridLayoutInvestimento5 = new QGridLayout(boxInvestimento3);

    fluxoCaixaWidget *fluxoCaixa = new fluxoCaixaWidget(boxInvestimento3);

    QPushButton *button = new QPushButton("Teste", boxInvestimento3);
    QVBoxLayout *vertical = new QVBoxLayout(boxInvestimento3);
    vertical->addWidget(button);
    vertical->addWidget(fluxoCaixa);

    //gridLayoutInvestimento5->addWidget(fluxoCaixa);

    gridLayoutInvestimento5->addLayout(vertical,0,0);
    boxInvestimento3->setLayout(gridLayoutInvestimento5);

    //------------------------------------------------------------------------

To copy to clipboard, switch view to plain text mode 

[d_stranz]

Your "fluxoCaixaWidget" is derived from QWidget, right? Have you reimplemented the QWidget methods that are used by layouts to determine what size your widget is supposed to be?

In particular, methods like QWidget::minimumSize(), QWidget::minimumSizeHint(), and QWidget::sizeHint()?