Bug 388592 - Be able to set keyboard shortcuts (fullscreen, rectangle etc.) in Spectacle
Summary: Be able to set keyboard shortcuts (fullscreen, rectangle etc.) in Spectacle
Status: RESOLVED FIXED
Alias: None
Product: Spectacle
Classification: Applications
Component: General (show other bugs)
Version: 17.12.1
Platform: Other Linux
: HI wishlist
Target Milestone: ---
Assignee: Boudhayan Gupta
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2018-01-05 19:23 UTC by Gregor Mi
Modified: 2019-06-21 15:41 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 19.08.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Gregor Mi 2018-01-05 19:23:24 UTC
It would be very helpful if the keyboard shortcuts for Spectacle could be reviewed directly in Spectacle.

Ideally, one would be able to also change the keyboard shortcuts from within Spectacle. Or have a link that opens the right place in the Plasma configuration: "Global Keyboard Shortcuts" > Component "KDE Daemon" (which is currently not intuitive to find).
Comment 1 David Redondo 2019-06-12 08:35:07 UTC
Some progress in https://phabricator.kde.org/D19310
I will try to progress this during the sprint
Comment 2 David Redondo 2019-06-21 15:41:37 UTC
Git commit 1d568009f73f3b63363511eddded623f16be86bb by David Redondo.
Committed on 21/06/2019 at 15:41.
Pushed by davidre into branch 'master'.

Port to KGlobalAccel

Summary:
Port to KGlobalAccel. This enables us to drop khotkeys and display a configuration
dialog inside the application. The update script correctly sets the defaults and migrates
possibly user changed shortcuts. I didn't know where to place the KActionCollection
and put it in SpectacleConfig for the moment.
FIXED-IN: 19.08.0

Test Plan: Shortcuts should work as before.

Reviewers: #spectacle, davidedmundson, ngraham

Reviewed By: #spectacle, davidedmundson, ngraham

Subscribers: asturmlechner, ngraham, #spectacle

Tags: #spectacle

Maniphest Tasks: T10519

Differential Revision: https://phabricator.kde.org/D19310

M  +2    -0    CMakeLists.txt
D  +0    -4    ExtraDesktop.sh
M  +15   -4    desktop/CMakeLists.txt
A  +115  -0    desktop/MigrateShortcuts.cpp     [License: UNKNOWN]  *
M  +5    -0    desktop/org.kde.spectacle.desktop
D  +0    -620  desktop/spectacle.khotkeys
A  +4    -0    desktop/spectacle_shortcuts.upd
M  +3    -0    src/CMakeLists.txt
M  +8    -0    src/Gui/SettingsDialog/SettingsDialog.cpp
A  +44   -0    src/Gui/SettingsDialog/ShortcutsOptionsPage.cpp     [License: UNKNOWN]  *
A  +31   -0    src/Gui/SettingsDialog/ShortcutsOptionsPage.h     [License: UNKNOWN]  *
M  +41   -0    src/SpectacleConfig.cpp
M  +4    -0    src/SpectacleConfig.h
M  +24   -1    src/SpectacleCore.cpp
M  +2    -0    src/SpectacleCore.h
M  +1    -1    tests/CMakeLists.txt

The files marked with a * at the end have a non valid license. Please read: https://community.kde.org/Policies/Licensing_Policy and use the headers which are listed at that page.


https://commits.kde.org/spectacle/1d568009f73f3b63363511eddded623f16be86bb