Qt show modal dialog (.ui) on menu item click

[tomorrow]

I have created several menu items in Qt but I do not know how to connect them to to the qtdialog which I created
for the menu items :
mainwindow.h

........
 
    void get_started_info();
 
private:
    Ui::MainWindow *ui;
    //! Create the menu classes
    void createActions();
    void createMenus();
    //! Menu Items
    QMenu   *home;
    QMenu   *help;
    QAction *quit;
    QAction *start;
 
};
 
#endif // MAINWINDOW_H

........

    void get_started_info();

private:
    Ui::MainWindow *ui;
    //! Create the menu classes
    void createActions();
    void createMenus();
    //! Menu Items
    QMenu   *home;
    QMenu   *help;
    QAction *quit;
    QAction *start;

};

#endif // MAINWINDOW_H

To copy to clipboard, switch view to plain text mode 

mainwindow.cpp

void MainWindow::createActions()
{
    QPixmap startpix("../src/img/start.png");
 
    start = new QAction(startpix,"&Get Started",this);
    start->setStatusTip(tr("Learn how to use this program"));
    connect(start,SIGNAL(triggered()),qApp,SLOT(get_started_info()));
 
}
 
void MainWindow::createMenus()
{
    home = menuBar()->addMenu(tr("&Home")); //! Adding a home item
    help = menuBar()->addMenu(tr("H&elp")); //! Adding a help item
    help->addAction(start); //! Adding About under the help
 
}

void MainWindow::createActions()
{
    QPixmap startpix("../src/img/start.png");

    start = new QAction(startpix,"&Get Started",this);
    start->setStatusTip(tr("Learn how to use this program"));
    connect(start,SIGNAL(triggered()),qApp,SLOT(get_started_info()));

}

void MainWindow::createMenus()
{
    home = menuBar()->addMenu(tr("&Home")); //! Adding a home item
    help = menuBar()->addMenu(tr("H&elp")); //! Adding a help item
    help->addAction(start); //! Adding About under the help

}

To copy to clipboard, switch view to plain text mode 

in the mainwindow.cpp I also have this class which should show the dialog

void MainWindow::get_started_info()
{
    get_started one(this); //create a object directly
    one.exec(); //modal dialog
}

void MainWindow::get_started_info()
{
    get_started one(this); //create a object directly
    one.exec(); //modal dialog
}

To copy to clipboard, switch view to plain text mode 

I also have get_started.h / get_started.cpp / get_strated.ui

I think there is an issue with this lines :

start = new QAction(startpix,"&Get Started",this);
    start->setStatusTip(tr("Learn how to use this program"));
    connect(start,SIGNAL(triggered()),qApp,SLOT(get_started_info()));

start = new QAction(startpix,"&Get Started",this);
    start->setStatusTip(tr("Learn how to use this program"));
    connect(start,SIGNAL(triggered()),qApp,SLOT(get_started_info()));

To copy to clipboard, switch view to plain text mode 

is it possible to show the dialog when the "Get Started" is clicked from the menu ?
thank you

[toufic.dbouk]

is it possible to show the dialog when the "Get Started" is clicked from the menu ?

Yes it is possible. In the the triggered Slot of the QAction you are using show the dialog you want.
For example:

void MainWindow::on_start_triggered()
{
one = new get_started(this); // make sure to have it as class member or use the exec() function if you create the instance in the slot without being a member.
one->setModal(true); // if you need it.
one->show();
}

void MainWindow::on_start_triggered()
{
one = new get_started(this); // make sure to have it as class member or use the exec() function if you create the instance in the slot without being a member.
one->setModal(true); // if you need it.
one->show();
}

To copy to clipboard, switch view to plain text mode 

connect(start,SIGNAL(triggered()),qApp,SLOT(get_started_info()));

connect(start,SIGNAL(triggered()),qApp,SLOT(get_started_info()));

To copy to clipboard, switch view to plain text mode 

why are you using qApp as a parent here ? your slot is in the current class so you should use this instead.

Good Luck.

[tomorrow]

thank you for your response

why are you using qApp as a parent here ? your slot is in the current class so you should use this instead.

sorry I was following a tutorial and thats how they did it, I changed it to "this"

Yes it is possible. In the the triggered Slot of the QAction you are using show the dialog you want.

I tried the code and had this :

void MainWindow::get_started_info()
{
    one = new get_started(this);
    one->setModal(true); // if you need it.
    one->show();
 
    //get_started one(this); //create a object directly
    //one.exec(); //modal dialog
}

void MainWindow::get_started_info()
{
    one = new get_started(this);
    one->setModal(true); // if you need it.
    one->show();

    //get_started one(this); //create a object directly
    //one.exec(); //modal dialog
}

To copy to clipboard, switch view to plain text mode 

and used this to connect :

start = new QAction(startpix,"&Get Started",this);
start->setStatusTip(tr("Learn how to use this program"));
connect(start,SIGNAL(triggered()),this,SLOT(get_started_info()));

start = new QAction(startpix,"&Get Started",this);
start->setStatusTip(tr("Learn how to use this program"));
connect(start,SIGNAL(triggered()),this,SLOT(get_started_info()));

To copy to clipboard, switch view to plain text mode 

but it says:
- "one" was not declared in this scope
- expected type-specified before 'get_started'
- expected ';' before get_started

here are the files for get_started

get_started.h :

#ifndef GET_STARTED_H
#define GET_STARTED_H
 
#include <QDialog>
#include "ui_get_started.h"
 
class get_started : public QDialog
{
    Q_OBJECT
public:
    explicit get_started(QWidget *parent = 0);
 
private:
    Ui:: get_started *ui;
};
 
 
#endif // GET_STARTED_H

#ifndef GET_STARTED_H
#define GET_STARTED_H

#include <QDialog>
#include "ui_get_started.h"

class get_started : public QDialog
{
    Q_OBJECT
public:
    explicit get_started(QWidget *parent = 0);

private:
    Ui:: get_started *ui;
};


#endif // GET_STARTED_H

To copy to clipboard, switch view to plain text mode 

get_started.cpp

#include "get_started.h"
#include "ui_get_started.h"
 
get_started::get_started(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::get_started)
{
 
    ui->setupUi(this);
}

#include "get_started.h"
#include "ui_get_started.h"

get_started::get_started(QWidget *parent) :
    QDialog(parent),
    ui(new Ui::get_started)
{

    ui->setupUi(this);
}

To copy to clipboard, switch view to plain text mode 

I have been trying to solve this issue for several hours now, really appreciate if you help me solve it

[toufic.dbouk]

Add a qDebug () in your get_started_info() slot to check if it is getting called when the triggered signal is fired. Try to create a QMessageBox to see if it will show.

QMessageBox::information(this,"title","message");// dont forget to include QMessageBox

QMessageBox::information(this,"title","message");// dont forget to include QMessageBox

To copy to clipboard, switch view to plain text mode 

You can go to the UI file in the designer and go to the action editor and right click your action , go to slot and press on triggered ( it will then automatically create a slot for you in your mainwindow ) and inside it create your dialog.
Did you create a class memember from your dialog ?

get_started *one; //in your mainwindow under private

get_started *one; //in your mainwindow under private

To copy to clipboard, switch view to plain text mode 

Good Luck.

[tomorrow]

void MainWindow::get_started_info()
{
    QMessageBox::information(this,"About","message");// dont forget to include QMessageBox
}

void MainWindow::get_started_info()
{
    QMessageBox::information(this,"About","message");// dont forget to include QMessageBox
}

To copy to clipboard, switch view to plain text mode 

this works, when I click on Get Started, this message box appears

get_started *one; //in your mainwindow under private

get_started *one; //in your mainwindow under private

To copy to clipboard, switch view to plain text mode 

like this :

private:
    Ui::MainWindow *ui;
    //! Create the menu classes
    void createActions();
    void createMenus();
    //! Menu Items
    QMenu   *home;
    QMenu   *help;
    QAction *quit;
    QAction *start;
    QAction *about;
    get_started *one;

private:
    Ui::MainWindow *ui;
    //! Create the menu classes
    void createActions();
    void createMenus();
    //! Menu Items
    QMenu   *home;
    QMenu   *help;
    QAction *quit;
    QAction *start;
    QAction *about;
    get_started *one;

To copy to clipboard, switch view to plain text mode 

but I get an error 'get_started' does not name a type

[toufic.dbouk]

but I get an error 'get_started' does not name a type

Did you include get_started.h ?
And you have

#include "ui_get_started.h"

#include "ui_get_started.h"

To copy to clipboard, switch view to plain text mode 

I am not sure if you need this to be included in both header and cpp file, maybe just in your .cpp file.

Good Luck.

[anda_skoa]

The original code was almost correct, the only mistake was usign qApp as the receiver object instead of this.

Creating the dialog on the stack and excuting it as a modal dialog using exec() where correct.

Cheers,
_

[toufic.dbouk]

Creating the dialog on the stack and excuting it as a modal dialog using exec() where correct.

Yes it was correct and we mentioned that he can keep his code since its right instead of creating it on heap. Creating it on the stack will limit the scope where you can use the object for further usage.