Problem in layout widgets in a grid

[franco.amato]

Hi to all,
layouts don't want enter in my mind.
I would layout 4 widgets in a grid ( 2 x 2 ), 2 QLabels and 2 QLeds.
I have some problems with the space between widgets and w and h of the layout.
I attach a screenshot layout.jpg.
Basically I layout some buttons horizontally having an height, then I add the gridlayout to the horizontally layout. The problem is that the height of the grid is bigger that the height of the buttons and is not what I would achieve. I would the gridlayout to be height as the buttons and also the horizontal space between widget is too big.
I tried with setHorizontalSpacing, setSpacing without success.
What can I do?
Regards

[franco.amato]

any reply please?

[franco.amato]

Well seems an unsolvable problem .
I hope to get a reply.
Best

[wysota]

Two rows of labels with this font are larger than the button. If you want them to fit, you need to decrease their size (and possibly the size of the other two widgets as well). You can also manipulate the spacing and margins of the layout but it won't do much without decreasing sizes of the widgets that are part of the layout.

[franco.amato]

Two rows of labels with this font are larger than the button. If you want them to fit, you need to decrease their size (and possibly the size of the other two widgets as well). You can also manipulate the spacing and margins of the layout but it won't do much without decreasing sizes of the widgets that are part of the layout.

Hi Wysota,
I followed your suggestion and moved the grid layout on the left but I still have a space that I can not remove.
I attach this screenshot spaces.jpg showing the space I would remove.

I post some code:

m_focusLed = new QLed(this); // I create a QLed
m_focusLed->setFixedSize(13,13);// set its size to 13, 13
m_encodingLed = new QLed(this); // the same for the other led
m_encodingLed->setFixedSize(13,13);
 
QLabel* activeLabel = new QLabel(tr("A")); // create a QLabel
activeLabel->setFixedSize(13,10); // set the size
QLabel* encodingLabel = new QLabel(tr("E"));
encodingLabel->setFixedSize(13, 10);
 
QGridLayout* gl = new QGridLayout(); // create the grid and add the widgets
gl->setAlignment(Qt::AlignLeft);
gl->setContentsMargins(0, 0, 0, 0);
gl->addWidget(activeLabel, 0, 0);
gl->addWidget(m_focusLed, 0, 1);
gl->addWidget(encodingLabel, 1, 0);
gl->addWidget(m_encodingLed, 1, 1);

m_focusLed = new QLed(this); // I create a QLed
m_focusLed->setFixedSize(13,13);// set its size to 13, 13
m_encodingLed = new QLed(this); // the same for the other led
m_encodingLed->setFixedSize(13,13);

QLabel* activeLabel = new QLabel(tr("A")); // create a QLabel
activeLabel->setFixedSize(13,10); // set the size
QLabel* encodingLabel = new QLabel(tr("E"));
encodingLabel->setFixedSize(13, 10);

QGridLayout* gl = new QGridLayout(); // create the grid and add the widgets
gl->setAlignment(Qt::AlignLeft);
gl->setContentsMargins(0, 0, 0, 0);
gl->addWidget(activeLabel, 0, 0);
gl->addWidget(m_focusLed, 0, 1);
gl->addWidget(encodingLabel, 1, 0);
gl->addWidget(m_encodingLed, 1, 1);

To copy to clipboard, switch view to plain text mode 

then I insert the grid layout in horizontal layout so:

/* main layout */
QHBoxLayout* lo = new QHBoxLayout();
lo->setSpacing(0);
lo->addLayout(gl); // <--add the grid layout
/* add more widgets ( buttons ) */
//...

/* main layout */
QHBoxLayout* lo = new QHBoxLayout();
lo->setSpacing(0);
lo->addLayout(gl); // <--add the grid layout
/* add more widgets ( buttons ) */
//...

To copy to clipboard, switch view to plain text mode 

Doing so I got such space and I would delete it.
I hope yo get help.

Best Regards

[wysota]

I did not suggest to move the layout anywhere.

[franco.amato]

I did not suggest to move the layout anywhere.

Yes I know, you suggested to me to decrease the size, and I did it. I did what you suggested AND I moved the layout on the left size

[wysota]

I don't see where exactly you decreased the font size. And your two leds plus the spacing between them are surely larger than the button that determines the height of the row.

[franco.amato]

I don't see where exactly you decreased the font size. And your two leds plus the spacing between them are surely larger than the button that determines the height of the row.

Hi Wysota,
the space I would remove is the horizontal space as indicated in the last screenshot I attached. It's located on the right of the leds.

[franco.amato]

Hi Wysota,
the space I would remove is the horizontal space as indicated in the last screenshot I attached. It's located on the right of the leds.

So how can I remove such space?
regards

[wysota]

I have no idea what you want. You show pictures with some things and write about other things.

[franco.amato]

I have no idea what you want. You show pictures with some things and write about other things.

Wysota the last screenshot I attached show the problem.
I attach it again spaces.jpg

Regards

[wysota]

Is the space caused by the layout or by the widgets in the layout?

[franco.amato]

Is the space caused by the layout or by the widgets in the layout?

Wysota I think by the layout, as I set the size of the widgets in the layout

[nroberts]

...the horizontal space between widget is too big.
I tried with setHorizontalSpacing, setSpacing without success.

I'm new to Qt so don't know how Qt does it, but every reasonably expressive UI API and sizer system allows you to tell an area within the sizer not to grow. I can't be sure, but my bet is that your grid is being expanded to fill the space given to it by the sizer it is within. You may need to tell THAT sizer that it shouldn't be giving any free space to your grid (or you might have to tell the grid not to take any, depending on how Qt does it). Then the space should be absorbed by a different widget.

Simply setting the size may very well be insufficient for this need. Sizers/layouts/whatever tend to give out space according to how THEY have been told to, letting the widgets given that space to resize as they desire, and then *positioning them within the space given* according to whatever rules have been dictated/defaulted.

It's just a guess, but it's a reasonable one and you're not getting any other answers...so there you go...

BTW, I tend to get quicker, better, possibly ANY answers at stackoverflow.com

[franco.amato]

I'm new to Qt so don't know how Qt does it, but every reasonably expressive UI API and sizer system allows you to tell an area within the sizer not to grow. I can't be sure, but my bet is that your grid is being expanded to fill the space given to it by the sizer it is within. You may need to tell THAT sizer that it shouldn't be giving any free space to your grid (or you might have to tell the grid not to take any, depending on how Qt does it). Then the space should be absorbed by a different widget.

Simply setting the size may very well be insufficient for this need. Sizers/layouts/whatever tend to give out space according to how THEY have been told to, letting the widgets given that space to resize as they desire, and then *positioning them within the space given* according to whatever rules have been dictated/defaulted.

It's just a guess, but it's a reasonable one and you're not getting any other answers...so there you go...

BTW, I tend to get quicker, better, possibly ANY answers at stackoverflow.com

Hi,
I think you're right but I have no idea on how to tell to the grid to don't get such space

[wysota]

Wysota I think by the layout, as I set the size of the widgets in the layout

Don't think. Check it out. See what size the widget wants to have and what size it indeed has.

[franco.amato]

Don't think. Check it out. See what size the widget wants to have and what size it indeed has.

I wrote this code:

m_focusLed = new QLed();
m_focusLed->setFixedSize(13,13);
m_focusLed->setSizePolicy(QSizePolicy::Fixed, QSizePolicy::Fixed);
QSize size = m_focusLed->sizeHint();
qDebug() << size.width() << " " << size.height();

m_focusLed = new QLed();
m_focusLed->setFixedSize(13,13);
m_focusLed->setSizePolicy(QSizePolicy::Fixed, QSizePolicy::Fixed);
QSize size = m_focusLed->sizeHint();
qDebug() << size.width() << " " << size.height();

To copy to clipboard, switch view to plain text mode 

And the result was -1 -1

[wysota]

Too bad

[franco.amato]

Too bad

Too bad my code or the result?

I reimplemented the sizeHint method for the QLed so:

QSize QLed::sizeHint() const
{
	return QSize(15, 15);
}

QSize QLed::sizeHint() const
{
	return QSize(15, 15);
}

To copy to clipboard, switch view to plain text mode 

But the space is still there so seems is a layout problem that definitively I don't know how to solve