Add QScrollArea to QDialog

[Rachol]

Ok, now it is easy

There are couple of ways to solve your problem.

1. The easiest way would be to place a layout inside your MixerStrip class and then place all the widgets inside the layout instead of hardcoding all coordinates.
2. Re-implement minimumSizeHint and maximumSizeHint. You could return the same value that you return in your sizeHint implementation.

[cpsmusic]

Ok, now it is easy

There are couple of ways to solve your problem.

1. The easiest way would be to place a layout inside your MixerStrip class and then place all the widgets inside the layout instead of hardcoding all coordinates.
2. Re-implement minimumSizeHint and maximumSizeHint. You could return the same value that you return in your sizeHint implementation.

I tried the second suggestion (see below) however the widgets are still spread out.

#include "mixerstrip.h"
 
MixerStrip::MixerStrip(QWidget *parent) :
    QWidget(parent)
{
    QSlider *volSlider = new QSlider(Qt::Vertical);
    volSlider->setMinimum(0);
    volSlider->setMaximum(100);
    QPalette pal = volSlider->palette();
    pal.setColor(volSlider->backgroundRole(), QColor(128, 128, 128));
    volSlider->setPalette(pal);
    volSlider->setAutoFillBackground(TRUE);
    volSlider->setGeometry(0,0,20,220);
    volSlider->setParent(this);
 
    Meter *meter = new Meter;
    meter->setGeometry(20,0,50,220);
    meter->setParent(this);
 
    QPushButton *muteButton = new QPushButton("M");
    QPushButton *soloButton = new QPushButton("S");
    muteButton->setCheckable(true);
    soloButton->setCheckable(true);
 
    pal = muteButton->palette();
    pal.setColor(muteButton->backgroundRole(), QColor(128, 128, 128));
    muteButton->setPalette(pal);
    muteButton->setAutoFillBackground(TRUE);
    pal = soloButton->palette();
    pal.setColor(soloButton->backgroundRole(), QColor(128, 128, 128));
    soloButton->setPalette(pal);
    soloButton->setAutoFillBackground(TRUE);
 
    muteButton->setGeometry(0,220,70,20);
    soloButton->setGeometry(0,240,70,20);
    muteButton->setParent(this);
    soloButton->setParent(this);
 
    QSlider *panSlider = new QSlider(Qt::Horizontal);
    panSlider->setMinimum(-100);
    panSlider->setMaximum(100);
    panSlider->setValue(0);
    pal = panSlider->palette();
    pal.setColor(panSlider->backgroundRole(), QColor(128, 128, 128));
    panSlider->setPalette(pal);
    panSlider->setAutoFillBackground(TRUE);
    panSlider->setGeometry(0,260,70,20);
    panSlider->setParent(this);
 
    QLineEdit *trackName = new QLineEdit();
    trackName->setAlignment(Qt::AlignCenter);
    trackName->setText("Track");
    trackName->setGeometry(0,280,70,20);
    trackName->setParent(this);
 
    QObject::connect(volSlider, SIGNAL(valueChanged(int)), meter, SLOT(valueChanged(int)));
    volSlider->setValue(0);
 
    setFixedSize(size());
    //setSizeConstraint ( QLayout::SetFixedSize );
}
 
QSize MixerStrip::sizeHint() const
{
    QSize size(70,300);
 
    return size;
}
 
QSize MixerStrip::minimumSizeHint() const
{
 
    QSize size(70,300);
 
    return size;
}
 
QSize MixerStrip::maximumSize()
{
    QSize size(70,300);
 
    return size;
}

#include "mixerstrip.h"

MixerStrip::MixerStrip(QWidget *parent) :
    QWidget(parent)
{
    QSlider *volSlider = new QSlider(Qt::Vertical);
    volSlider->setMinimum(0);
    volSlider->setMaximum(100);
    QPalette pal = volSlider->palette();
    pal.setColor(volSlider->backgroundRole(), QColor(128, 128, 128));
    volSlider->setPalette(pal);
    volSlider->setAutoFillBackground(TRUE);
    volSlider->setGeometry(0,0,20,220);
    volSlider->setParent(this);

    Meter *meter = new Meter;
    meter->setGeometry(20,0,50,220);
    meter->setParent(this);

    QPushButton *muteButton = new QPushButton("M");
    QPushButton *soloButton = new QPushButton("S");
    muteButton->setCheckable(true);
    soloButton->setCheckable(true);

    pal = muteButton->palette();
    pal.setColor(muteButton->backgroundRole(), QColor(128, 128, 128));
    muteButton->setPalette(pal);
    muteButton->setAutoFillBackground(TRUE);
    pal = soloButton->palette();
    pal.setColor(soloButton->backgroundRole(), QColor(128, 128, 128));
    soloButton->setPalette(pal);
    soloButton->setAutoFillBackground(TRUE);

    muteButton->setGeometry(0,220,70,20);
    soloButton->setGeometry(0,240,70,20);
    muteButton->setParent(this);
    soloButton->setParent(this);

    QSlider *panSlider = new QSlider(Qt::Horizontal);
    panSlider->setMinimum(-100);
    panSlider->setMaximum(100);
    panSlider->setValue(0);
    pal = panSlider->palette();
    pal.setColor(panSlider->backgroundRole(), QColor(128, 128, 128));
    panSlider->setPalette(pal);
    panSlider->setAutoFillBackground(TRUE);
    panSlider->setGeometry(0,260,70,20);
    panSlider->setParent(this);

    QLineEdit *trackName = new QLineEdit();
    trackName->setAlignment(Qt::AlignCenter);
    trackName->setText("Track");
    trackName->setGeometry(0,280,70,20);
    trackName->setParent(this);

    QObject::connect(volSlider, SIGNAL(valueChanged(int)), meter, SLOT(valueChanged(int)));
    volSlider->setValue(0);

    setFixedSize(size());
    //setSizeConstraint ( QLayout::SetFixedSize );
}

QSize MixerStrip::sizeHint() const
{
    QSize size(70,300);

    return size;
}

QSize MixerStrip::minimumSizeHint() const
{

    QSize size(70,300);

    return size;
}

QSize MixerStrip::maximumSize()
{
    QSize size(70,300);

    return size;
}

To copy to clipboard, switch view to plain text mode 

[Santosh Reddy]

Ok, I hope this should fix your problem for sure.

When you implement size hints on your MixerStrip, then you don't need to set it to fixed size explicitly. Just remove the following statement at the end of MixerStrip Ctor, you should be done, worrying.

//setFixedSize(size());

//setFixedSize(size());

To copy to clipboard, switch view to plain text mode 

[cpsmusic]

Thanks, that last change solved the problem! Thanks very much, the people on this forum are very helpful.