Bug 309335 - KSnapshot should not bind "new screenshot" to the space key
Summary: KSnapshot should not bind "new screenshot" to the space key
Status: RESOLVED UNMAINTAINED
Alias: None
Product: ksnapshot
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Debian testing Linux
: NOR minor
Target Milestone: ---
Assignee: Richard Moore
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-31 18:28 UTC by Tenno Seremel
Modified: 2018-09-04 08:44 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch that removes this line. (530 bytes, patch)
2012-10-31 18:36 UTC, Tenno Seremel
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tenno Seremel 2012-10-31 18:28:32 UTC
In a ksnapshot.cpp KSnapshot binds Space key to a "new screenshot" button via QShortcut() call. This breaks user's expectation that pressing Space means "activate this button."

Reproducible: Always

Steps to Reproduce:
1. Run ksnapshot.
2. Press tab a few times until you get to save file button.
3. Press Space.
Actual Results:  
Dialog disappears and screenshotting mode is activated.

Expected Results:  
Save file dialog should appear.

Qt: 4.8.2
KDE: 4.8.4 (4.8.4)
Comment 1 Tenno Seremel 2012-10-31 18:36:25 UTC
Created attachment 74903 [details]
Patch that removes this line.
Comment 2 Vladimir Panteleev 2016-01-01 21:54:43 UTC
Agreed. This behavior/code is actually redundant because the "Take a New Snapshot" button is already focused by default, thus Space will continue working as a "new snapshot" key until the user changes focus.
Comment 3 Sasha Unspecified 2018-07-06 14:21:45 UTC
I confirm this for 0.8.2. Pressing Space when "Send to..." is focused causes starting a new snapshot instead of opening the "Send to..." menu.
Comment 4 Sasha Unspecified 2018-07-06 14:37:52 UTC
BTW, isn't this bug a one that causes #391066?
Comment 5 Sasha Unspecified 2018-07-06 14:41:05 UTC
I mean that bug 391066 may be caused by the fact that due to binding "new snapshot" to Space pressing Space when the "new snapshot" is focused causes the "new snapshot" action to be initiated twice.
Comment 6 Christoph Feck 2018-07-06 14:49:49 UTC
Sasha, KSnapshot is long unmaintained. Please use 'Spectacle' or any other maintained screenshot utility.
Comment 7 Sasha Unspecified 2018-07-06 15:18:41 UTC
Christoph Feck, thanks, I didn't know about that software.
Comment 8 Andrew Crouthamel 2018-09-04 04:17:31 UTC
Hello! Sorry to be the bearer of bad news, but this project has been unmaintained for many years and I will be closing this bug. Spectacle is the replacement for ksnapshot now. Please test again and file a new bug for Spectacle if you still have issues. Thank you!
Comment 9 Sasha Unspecified 2018-09-04 08:44:12 UTC
That's actually good news :).