Bug 309335

Summary:	KSnapshot should not bind "new screenshot" to the space key
Product:	[Unmaintained] ksnapshot	Reporter:	Tenno Seremel <mystra.x64>
Component:	general	Assignee:	Richard Moore <rich>
Status:	RESOLVED UNMAINTAINED
Severity:	minor	CC:	adaptee, arthur, kde, sasha2048
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Patch that removes this line.

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 :).