closing a dialog

[axeljaeger]

Header looks OK. The only thing I do not like is a lowercase class name because usually the first letter in the classname is uppercase.

Regarding the dialog: Create the constructor in the constructor, save the pointer in a member variable and then only call show on it in your method.

[chrisb123]

Regarding the dialog: Create the constructor in the constructor, save the pointer in a member variable and then only call show on it in your method.

how do i save the pointer in a member variable?
i assume you mean "web"
and how do i only call show in the method?
I want the widget to display immediately after selecting the menu item

so it should like this
the slot function gets called from a menu item

void test::web() {
  webapp::webapp(*parent);

void test::web() {
  webapp::webapp(*parent);

To copy to clipboard, switch view to plain text mode 

which calls this

webapp::webapp(QWidget* parent) {
 QWidget *web = new webapp(this);
  web->resize(1000, 350);
  web->show();
  view = new QWebView(this);
  view->show();
}

webapp::webapp(QWidget* parent) {
 QWidget *web = new webapp(this);
  web->resize(1000, 350);
  web->show();
  view = new QWebView(this);
  view->show();
}

To copy to clipboard, switch view to plain text mode 

It doesnt work, the program runs but the new widget doesnt display

[axeljaeger]

Yes, because both widget gets "this" as parent. If you want your webview to be shown inside "web", pass web as the parent to the webview.

Then do not use QWidget as baseclass for web but QDialog so it will be shown in a new window on the screen.

[chrisb123]

Still does not bring up a new dialog

class webapp : public QDialog {
  Q_OBJECT
  public:
    webapp(QDialog* parent);
.............
void test::web() {
  webapp::webapp(*parent);
}
webapp::webapp(QDialog* parent) {
 QDialog *web = new webapp(this);
  web->show();
  view = new QWebView(web);

class webapp : public QDialog {
  Q_OBJECT
  public:
    webapp(QDialog* parent);
.............
void test::web() {
  webapp::webapp(*parent);
}
webapp::webapp(QDialog* parent) {
 QDialog *web = new webapp(this);
  web->show();
  view = new QWebView(web);

To copy to clipboard, switch view to plain text mode 

[axeljaeger]

Please find the attached example.

[chrisb123]

thanks, i'll have to study this tonight at work

[chrisb123]

I dont have main because it is a plasma widget
It's setup like the example you gave me

mum::mum(QObject *parent, const QVariantList &args)
    : Plasma::Applet(parent, args),
    m_svg(this),
    m_icon("document"),
    m_webapp(NULL),
    m_setapp(NULL)
{
    m_webapp = new webapp(this);
    m_setapp = new setapp();
}
void mum::settings() { 
 m_setapp->show();
 
}
void mum::web() {
  m_webapp->show();
}

mum::mum(QObject *parent, const QVariantList &args)
    : Plasma::Applet(parent, args),
    m_svg(this),
    m_icon("document"),
    m_webapp(NULL),
    m_setapp(NULL)
{
    m_webapp = new webapp(this);
    m_setapp = new setapp();
}
void mum::settings() { 
 m_setapp->show();

}
void mum::web() {
  m_webapp->show();
}

To copy to clipboard, switch view to plain text mode 

setapp works as expected
webapp gives the following error

mum.cpp: In constructor 'mum::mum(QObject*, const QVariantList&)':
mum.cpp:30: error: no matching function for call to 'webapp::webapp(mum* const)'
webapp.h:33: note: candidates are: webapp::webapp(QWidget*)

mum.cpp: In constructor 'mum::mum(QObject*, const QVariantList&)':
mum.cpp:30: error: no matching function for call to 'webapp::webapp(mum* const)'
webapp.h:33: note: candidates are: webapp::webapp(QWidget*)

To copy to clipboard, switch view to plain text mode 

[axeljaeger]

Looks like Plasma::Applet does not inherit QWidget. OK, then you can't have that as parent, just pass NULL as parent and delete webapp in your destructor.

[chrisb123]

how would i make it show the exisitng m_webapp from m_setapp?

[axeljaeger]

Either create m_webapp inside m_setapp or have m_setapp emit a signal and connect that to the show-slot of m_webapp.

[chrisb123]

Either create m_webapp inside m_setapp or have m_setapp emit a signal and connect that to the show-slot of m_webapp.

I'll have to read up on that

I want the widget to not create the web app utill its called for e.g.

void mum::web() {
  if ( ! m_webapp ) {
    m_webapp = new webapp(NULL);
  }
  m_webapp->show();
}

void mum::web() {
  if ( ! m_webapp ) {
    m_webapp = new webapp(NULL);
  }
  m_webapp->show();
}

To copy to clipboard, switch view to plain text mode 

How do I destroy m_webapp after closing it?

[axeljaeger]

Just delete it in the destructor. (once in the lifetime of your application).