ctrl+q shortcut key does not work 0.4.1

[William Mitchell Jr]

Ctrl+Q shortcut key does not work.
built with Qt 4.8.4 from qpdfview-0.4.1.tar.gz

In
mainwindow.cpp
if
m_exitAction->setShortcut(QKeySequence::Quit);
is replaced with
m_exitAction->setShortcut( tr("Ctrl+Q"));
it works.

I'm on
Debian/Sid
KDE 4.8.4
kernel 3.2.39-2 i686

[Adam Reichold (adamreichold)]

Hello William,

Thanks for your report. The constant "QKeySequence::Quit" is a context sensitive keyboard shortcut that should map to Ctrl+Q when running KDE. But I am not sure we can diagnose the problem we there observations alone.

Is the shortcut "Ctrl+Q" displayed in the menu? Have you tried setting "Ctrl+Q" in the shortcut editor in the settings dialog? Do any other shortcuts seem inoperable? (Last but not least, if you build from source: Have you tried building the current release 0.4.2 or even a snapshot from trunk? There are several changes to keyboard handling done in those. Or maybe try Benjamin's next package at [1].)

Best regards, Adam.

[1] http://mentors.debian.net/package/qpdfview

[William Mitchell Jr (wdmjun)]

Adam,
The "Ctrl+Q" is not displayed when compiled from source,
but "Ctrl-Q" is displayed when compiled after my patch is applied.
All other shortcuts appear to be working (but I have not tested all of them
yet).
I'll try the snapshot from trunk.
WM

On Fri, Apr 26, 2013 at 12:45 AM, Adam Reichold <email address hidden>wrote:

> Hello William,
>
> Thanks for your report. The constant "QKeySequence::Quit" is a context
> sensitive keyboard shortcut that should map to Ctrl+Q when running KDE.
> But I am not sure we can diagnose the problem we there observations
> alone.
>
> Is the shortcut "Ctrl+Q" displayed in the menu? Have you tried setting
> "Ctrl+Q" in the shortcut editor in the settings dialog? Do any other
> shortcuts seem inoperable? (Last but not least, if you build from
> source: Have you tried building the current release 0.4.2 or even a
> snapshot from trunk? There are several changes to keyboard handling done
> in those. Or maybe try Benjamin's next package at [1].)
>
> Best regards, Adam.
>
> [1] http://mentors.debian.net/package/qpdfview
>
> ** Changed in: qpdfview
> Status: New => Incomplete
>
> ** Changed in: qpdfview
> Importance: Undecided => Medium
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1172980
>
> Title:
> ctrl+q shortcut key does not work 0.4.1
>
> Status in qpdfview:
> Incomplete
>
> Bug description:
> Ctrl+Q shortcut key does not work.
> built with Qt 4.8.4 from qpdfview-0.4.1.tar.gz
>
> In
> mainwindow.cpp
> if
> m_exitAction->setShortcut(QKeySequence::Quit);
> is replaced with
> m_exitAction->setShortcut( tr("Ctrl+Q"));
> it works.
>
> I'm on
> Debian/Sid
> KDE 4.8.4
> kernel 3.2.39-2 i686
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qpdfview/+bug/1172980/+subscriptions
>

[Adam Reichold (adamreichold)]

Merged this with bug #1174119 to reduce tracking overhead as this is probably the same issue.