Assigning event filter for widgets inside groupbox

[mania]

Hi All,
I have used three push buttons inside a groupbox and when I assign eventFilter() for the pushbuttons it gets hidden in the output screen. groupbox has NoFocus policy and pushbuttons has StrongFocus policy.

test1::test1(QWidget *parent) :
    QDialog(parent, Qt::FramelessWindowHint),
    ui(new Ui::test1)
{
    ui->setupUi(this);
    ui->pushButton_1->installEventFilter(this);
    ui->pushButton_2->installEventFilter(this);
    ui->pushButton_3->installEventFilter(this);
}

test1::test1(QWidget *parent) :
    QDialog(parent, Qt::FramelessWindowHint),
    ui(new Ui::test1)
{
    ui->setupUi(this);
    ui->pushButton_1->installEventFilter(this);
    ui->pushButton_2->installEventFilter(this);
    ui->pushButton_3->installEventFilter(this);
}

To copy to clipboard, switch view to plain text mode 

bool test1::eventFilter(QObject *o, QEvent *e)
{
    if(e->type()==QEvent::KeyPress)
    {
        return QObject::eventFilter(o,e);
    }
}

bool test1::eventFilter(QObject *o, QEvent *e)
{
    if(e->type()==QEvent::KeyPress)
    {
        return QObject::eventFilter(o,e);
    }
}

To copy to clipboard, switch view to plain text mode 

[stampede]

Pay atention to compiler warnings, it should warn you that there is no return value in function returning bool:

bool test1::eventFilter(QObject *o, QEvent *e){
    if(e->type()==QEvent::KeyPress){
       return QObject::eventFilter(o,e);
    }
    // and if not keyPress then ... ?
}

bool test1::eventFilter(QObject *o, QEvent *e){
    if(e->type()==QEvent::KeyPress){
       return QObject::eventFilter(o,e);
    }
    // and if not keyPress then ... ?
}

To copy to clipboard, switch view to plain text mode 

Result returned by such function is undefined. Probably you are filtering out the paint events for these widgets.
To fix this, always return default base class eventFilter() when you don't want to filter the event out:

bool test1::eventFilter(QObject *o, QEvent *e){
    if(e->type()==QEvent::KeyPress){
       // handle key press
       return true; // filter out this event
    }
    return QDialog::eventFilter(o,e); // or whatever is the base class for test1
}

bool test1::eventFilter(QObject *o, QEvent *e){
    if(e->type()==QEvent::KeyPress){
       // handle key press
       return true; // filter out this event
    }
    return QDialog::eventFilter(o,e); // or whatever is the base class for test1
}

To copy to clipboard, switch view to plain text mode 

To avoid such errors in the future, add this flag to compiler flags:

-Werror=return-type

-Werror=return-type

To copy to clipboard, switch view to plain text mode 

or better

-Wall -Werror

-Wall -Werror

To copy to clipboard, switch view to plain text mode 

[anda_skoa]

Your event filter is not returning for all cases, so the show or paint events are likely filtered out before they reach the buttons.

What is it that you want to achieve?

Cheers,
_

[d_stranz]

Pay attention to compiler warnings, it should warn you that there is no return value in function returning bool

Interesting. Visual C++ will not let you get away with this - it's a compilation error, not a warning. All paths must return a valid value when the return type is not void.