From kfm-devel Sat Jun 15 05:55:12 2002 From: Doug Hanley Date: Sat, 15 Jun 2002 05:55:12 +0000 To: kfm-devel Subject: Re: [PATCH] Warning on close X-MARC-Message: https://marc.info/?l=kfm-devel&m=102412062602664 On Friday 14 June 2002 08:58 pm, Waldo Bastian wrote: > On Friday 14 June 2002 05:25 pm, Doug Hanley wrote: > > Hey, > > > > How does this look for a warning on close patch? > > I think it should be moved into the following if: > > if ( static_cast(kapp)->closedByUser() ) > > No need to generate this warning on logout IMO. > Ok, how's this? =2D- Doug Index: konq_mainwindow.cc =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /home/kde/kdebase/konqueror/konq_mainwindow.cc,v retrieving revision 1.1013 diff -u -3 -p -r1.1013 konq_mainwindow.cc =2D-- konq_mainwindow.cc 14 Jun 2002 04:46:41 -0000 1.1013 +++ konq_mainwindow.cc 15 Jun 2002 05:43:53 -0000 @@ -3826,13 +3839,24 @@ void KonqMainWindow::updateBookmarkBar() void KonqMainWindow::closeEvent( QCloseEvent *e ) { kdDebug(1202) << "KonqMainWindow::closeEvent begin" << endl; + // This breaks session management (the window is withdrawn in kwin) // so let's do this only when closed by the user. if ( static_cast(kapp)->closedByUser() ) { + if ( viewManager()->docContainer() &&=20 viewManager()->docContainer()->frameType()=3D=3D"Tabs" ) + { + if ( KMessageBox::warningYesNo( 0L, "You have multiple tabs open in= =20 this window, are you sure you wish to close it?", "Confirmation" ) =3D=3D=20 KMessageBox::No ) + { + e->ignore(); + return; + } + } + hide(); qApp->flushX(); } + KParts::MainWindow::closeEvent( e ); kdDebug(1202) << "KonqMainWindow::closeEvent end" << endl; }