Bug 357325

Summary:	GUI options are not saved on exit; also editing ~/.config/spectaclerc does not affect GUI options at next launch
Product:	[Applications] Spectacle	Reporter:	E. Hakan Duran <ehakanduran>
Component:	General	Assignee:	Boudhayan Gupta <me>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	ehakanduran, vkrevs
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	http://commits.kde.org/kscreengenie/7a4fa9297d2ef9dd696b92fe4d28023993872b2e	Version Fixed In:	16.04
Sentry Crash Report:

Description E. Hakan Duran 2015-12-30 04:49:03 UTC

I would like the "On Click" checkbox to remain selected in Spectacle. However, there doesn't seem to be a way to accomplish this. Spectacle won't remember this selection on exit. Additionally editing its configuration file stated above (converting waitCaptureOnClick=false to waitCaptureOnClick=true) does not have any impact on the next launch.

Reproducible: Always

Steps to Reproduce:
1. Launch Spectacle
2. Click to select "On Click" checkbox
3. Close the Spectacle window after saving the screenshot
4. Re-launch Spectacle
5. Find "On Click" checkbox unchecked


Expected Results:  
There should either be a settings option on GUI or, Spectacle should respect the spectaclerc file and behave accordingly.

Comment 1 Boudhayan Gupta 2015-12-30 09:05:52 UTC

I can reproduce this issue. Target fix 15.12.1

Comment 2 E. Hakan Duran 2015-12-31 21:26:59 UTC

Thanks!

Comment 3 Boudhayan Gupta 2016-01-11 19:13:20 UTC

I'm postponing the fix to 16.04 - there are a few related bugs with how the configuration is handled and what I'm going to do will fix them all.

Hope this isn't a showstopper for you for the time being.

Comment 4 E. Hakan Duran 2016-01-12 05:08:40 UTC

No, it is not a showstopper at all. Thank you for the update!

Comment 5 Boudhayan Gupta 2016-01-16 18:02:07 UTC

Git commit 7a4fa9297d2ef9dd696b92fe4d28023993872b2e by Boudhayan Gupta.
Committed on 16/01/2016 at 17:37.
Pushed by bgupta into branch 'master'.

Move all checkbox state saves to SpectacleConfig
FIXED-IN: 16.04

M  +15   -24   src/Gui/KSWidget.cpp
M  +0    -1    src/Gui/KSWidget.h
M  +52   -0    src/SpectacleConfig.cpp
M  +13   -1    src/SpectacleConfig.h

http://commits.kde.org/kscreengenie/7a4fa9297d2ef9dd696b92fe4d28023993872b2e