Button inherit stylesheet for no reason

J.Hilk

@JonB

What is sender() at the point this is initialized?

as it's a qobject_cast and doesn't crash on start up, most likely a nullptr?

What is the value of last_button the very first time this function gets called?

as it's not checked against 0, it should fail. Maybe luck?

using sender() is bad design anyway. One shouldn't use it if possible.

@mrjj posted a nice qt alternative I would suggest the OP to use that ;)

legitnameyo

How do I implement this code way? I do my UI in code since that's what I prefer. This is what I've got so far and it isn't working...

QButtonGroup *g_btns = new QButtonGroup(this);
/* all buttons are created and I add the buttons in about this way*/
btn1->setCheckable(true);
//etc..
g_btns->addButton(btn1);
g_btns->addButton(btn2);
g_btns->addButton(btn3);
// etc ...

      connect(g_btns, SIGNAL(buttonClicked(int)), this, SLOT(onGroupButtonClicked(int)));

void MainWindow::onGroupButtonClicked(int btn_id) {
qDebug() << "clicked!";
}

And this gives me nothing. The onGroupButtonClicked does not activate.

J.Hilk

@legitnameyo I never used QButtonGroup, but a quick look into the documentation tells me, you never gave your added buttons an ID

void QButtonGroup::addButton(QAbstractButton *button, int id = -1)

I'm guessing here and say, that buttons with the ID of -1 do not emit the buttonClicked(int) signal

jsulm

@legitnameyo It should work. Do you get any warnings at runtime (related to connect)?
Is g_btns in connect() really the one you're creating at the beginning of the code snippet you provided?

mrjj

@J.Hilk
Was a good guess,
but it seems, it will use negative values for auto numbering and does, in fact, emit them.

  QButtonGroup *buttonGroup = new QButtonGroup(this);

    buttonGroup->addButton( ui->pushButton_2);
    buttonGroup->addButton( ui->pushButton_3);
    buttonGroup->addButton( ui->pushButton_4);
    buttonGroup->addButton( ui->pushButton_5);


    connect(buttonGroup, static_cast<void(QButtonGroup::*)(int)>(&QButtonGroup::buttonClicked),
    [ = ](int id) {
        qDebug() << id;
    });

outputs.
-2
-2
-3
-5
-4

J.Hilk

@mrjj
I find this behavior actually not very intuitive.

That the id increases when non is specified, maybe...
But a higher negative number !?

mrjj

@J.Hilk
Yep. I had to test it as sounded odd it would auto number with negative values.
But it seems design wants to allow to have some auto numbered and some manually
even i don't see why.

JonB

@mrjj
Negative-decrementing auto-numbers are usually used for "anonymous" items while allowing positive numbers for explicit use. Then the numbers won't clash.

legitnameyo

There is NO error such as this one below

QMetaObject::connectSlotsByName: No matching signal for onGroupButtonClicked(int)

And yes g_btns is defined only ONCE. Everything is in the order I got it in my code. However one thing that might change things up is that I use a custom class for my buttons called QClickAgnostic, which inherits from QPushButton as such above

and the buttons are created like this

        QClickAgnostic *btn = new QClickAgnostic(ui->frame_of_btns);

KillerSmath

@legitnameyo
You need to notify the ButtonGroup about the click event.

I took a look at the source code of the abstractbutton and noticed the button directly notifies the group that it was clicked. But you broke this step when you reimplemented the MouseReleaseEvent function.

See a snippet of this step.

void QAbstractButton::mouseReleaseEvent(QMouseEvent *e)
{
...
    if (hitButton(e->pos())) {
        d->repeatTimer.stop();
        d->click(); // call click function 
        e->accept();
    } else {
        setDown(false);
        e->ignore();
    }
}
void QAbstractButtonPrivate::click()
{
    ...
    if (guard)
        emitReleased();
    if (guard)
        emitClicked(); // call emitClicked
}

void QAbstractButtonPrivate::emitClicked()
{
  ...
#if QT_CONFIG(buttongroup)
    if (guard && group) {
        emit group->buttonClicked(group->id(q)); // emit the signal
        if (guard && group)
            emit group->buttonClicked(q); // emit the signal
    }
#endif
}

So, you can solve this problem calling the click() function inside MouseReleaseEvent.

void QClickAgnostic::mouseReleaseEvent(QMouseEvent *e) {
    if(e->button() == Qt::LeftButton) {
        click(); // called
        emit leftClicked();
    } else if(e->button() == Qt::RightButton) {
        click(); // called
        emit rightClicked();
    }
}

mrjj

@KillerSmath
Good digging !

legitnameyo

@KillerSmath fix did the work, combined with @mrjj QGroupButton! I added this code to style the pressed button

void MainWindow::onGroupButtonClicked(int btn_id) {
    g_btns->button(btn_id)->setStyleSheet(QString("QPushButton {color: #444444; background: #cccccc; border: 2px solid #cccccc; border-width: 0px 0px 2px 0px;} QPushButton:hover {border: 2px black solid; color: #333333; background-color: #F7F7F7;} QPushButton:pressed {background: #cccccc;}"));
}

Thanks a lot guys!