[cmake-developers] [PATCH] QtDialog: Set Ctrl+Q as the shortcut for quitting the program.
Clinton Stimpson
clinton at elemtech.com
Thu Dec 15 10:27:59 EST 2011
On Thursday, December 15, 2011 08:20:14 am Raphael Kubo da Costa wrote:
> Clinton Stimpson <clinton at elemtech.com> writes:
> > On Tuesday, December 13, 2011 04:51:16 pm Raphael Kubo da Costa wrote:
> >> QKeySequence::Quit does not work on all platforms, and since it
> >> translates to Ctrl+Q on all platforms where it does work, Ctrl+Q was
> >> hardcoded instead.
> >> ---
> >>
> >> Source/QtDialog/CMakeSetupDialog.cxx | 2 ++
> >> 1 files changed, 2 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/Source/QtDialog/CMakeSetupDialog.cxx
> >> b/Source/QtDialog/CMakeSetupDialog.cxx index 1c058d3..338eaff 100644
> >> --- a/Source/QtDialog/CMakeSetupDialog.cxx
> >> +++ b/Source/QtDialog/CMakeSetupDialog.cxx
> >> @@ -26,6 +26,7 @@
> >>
> >> #include <QMimeData>
> >> #include <QUrl>
> >> #include <QShortcut>
> >>
> >> +#include <QKeySequence>
> >>
> >> #include <QMacInstallDialog.h>
> >>
> >> #include "QCMake.h"
> >>
> >> @@ -99,6 +100,7 @@ CMakeSetupDialog::CMakeSetupDialog()
> >>
> >> QObject::connect(this->DeleteCacheAction, SIGNAL(triggered(bool)),
> >>
> >> this, SLOT(doDeleteCache()));
> >>
> >> this->ExitAction = FileMenu->addAction(tr("E&xit"));
> >>
> >> + this->ExitAction->setShortcut(QKeySequence(Qt::CTRL + Qt::Key_Q));
> >>
> >> QObject::connect(this->ExitAction, SIGNAL(triggered(bool)),
> >>
> >> this, SLOT(close()));
> >
> > Can you instead make that
> > + this->ExitAction->setShortcut(QKeySequence::Quit);
> >
> > Do you want to redo the patch, or shall we just make the change to use
> > Quit?
>
> As I mentioned in the commit message, QKeySequence::Quit will not work
> on all platforms (the documentation says it translates to nothing on
> Windows, for example, and I couldn't use it on the i3 window manager,
> for example). I can change it if you prefer, though.
Ok, the patch is fine.
Is this something that needs brought up to the Qt developer's attention?
--
Clinton Stimpson
Elemental Technologies, Inc
Computational Simulation Software, LLC
www.csimsoft.com
More information about the cmake-developers
mailing list