Bug 357679

Summary: Not enough shortcut key options
Product: [Applications] trojita Reporter: Luke-Jr <luke-jr+kdebugs>
Component: Desktop GUIAssignee: Trojita default assignee <trojita-bugs>
Status: RESOLVED FIXED    
Severity: wishlist    
Priority: NOR    
Version First Reported In: git   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Luke-Jr 2016-01-07 23:33:03 UTC
I like to use left/right to navigate between previous/next (read or unread) messages, and Esc to close popup windows. These don't seem to be available for configuring.

Reproducible: Always
Comment 1 Thomas Lübking 2016-01-08 10:01:12 UTC
> Esc to close popup windows. 
?
That's a system feature - and should actually work on "linux" (whether you mean popup menus or modal dialogs - latter are canceled by esc)

What window are you talking about in particular and on which desktop environment (ie. Qt QPA)
Comment 2 Luke-Jr 2016-01-08 16:13:43 UTC
To be clear I only use it for, and only want "Esc" to close specifically the pop-up "view email" window. I definitely do not want to to close other windows in general.
Comment 3 Thomas Lübking 2016-01-08 19:28:04 UTC
https://gerrit.vesnicky.cesnet.cz/r/538
Comment 4 Jan Kundrát 2016-02-04 14:05:28 UTC
Git commit d45708b884f1948279e6505dc80676baca6385e6 by Jan Kundrát.
Committed on 02/02/2016 at 00:22.
Pushed by gerrit into branch 'master'.

GUI: Add an action for closing of standalone message windows

Unfortunately, one cannot use the GUI to assign for example "Esc" for
this particular shortcut because the GUI settings dialog intercepts that
one, and closes itself. However, assigning some other key for this
action and editing the config file afterwards works.

I've also double-checked that the signal-slots are correct, and that
only one popup window is closed when pressing that shortcut.

This is an alternative to https://gerrit.vesnicky.cesnet.cz/r/538 .

Change-Id: I7479be102736f071321fb7132a7f688f95c4bd7c

M  +3    -0    src/Gui/Window.cpp

http://commits.kde.org/trojita/d45708b884f1948279e6505dc80676baca6385e6
Comment 5 Jan Kundrát 2016-11-24 08:59:55 UTC
I think that the commit listed in comment #4 is as good as one can make it -> closing. Please reopen if you disagree.