After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 673003 - Add back the option to only show the save dialog
Add back the option to only show the save dialog
Status: RESOLVED DUPLICATE of bug 669629
Product: gnome-screenshot
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-screenshot-maint
gnome-screenshot-maint
Depends on:
Blocks:
 
 
Reported: 2012-03-28 15:15 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2012-05-08 16:25 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Add back the option to only show the save dialog (3.25 KB, patch)
2012-03-28 15:15 UTC, Jasper St. Pierre (not reading bugmail) 		none Details | Review
Update .gitignore (840 bytes, patch)
2012-03-28 15:15 UTC, Jasper St. Pierre (not reading bugmail) 		committed Details | Review

Description Jasper St. Pierre (not reading bugmail) 2012-03-28 15:15:35 UTC 
See patches.
Comment 1 Jasper St. Pierre (not reading bugmail) 2012-03-28 15:15:37 UTC 
Created attachment 210787 [details] [review]
Add back the option to only show the save dialog
Comment 2 Jasper St. Pierre (not reading bugmail) 2012-03-28 15:15:39 UTC 
Created attachment 210788 [details] [review]
Update .gitignore
Comment 3 Matthias Clasen 2012-03-28 23:49:41 UTC 
Doesn't make sense to me.
How is --interactive not good enough ?
You can just change the keybinding to use that.

Also, it appears to me that your patch breaks --interactive
Comment 4 Jasper St. Pierre (not reading bugmail) 2012-03-29 00:01:57 UTC 
(In reply to comment #3)
> Doesn't make sense to me.
> How is --interactive not good enough ?

No. I don't want the initial dialog asking me if I want area/window screenshot -- I already chose with which keybinding I pressed.

> You can just change the keybinding to use that.

By patching gnome-settings-daemon? Seems to be hardcoded in there.

> Also, it appears to me that your patch breaks --interactive

Why?
Comment 5 Matthias Clasen 2012-03-29 16:06:31 UTC 
(In reply to comment #4)

> 
> > Also, it appears to me that your patch breaks --interactive
> 
> Why?

-  if (screenshot_config->interactive)
+  if (screenshot_config->show_save_dialog)
Comment 6 Jasper St. Pierre (not reading bugmail) 2012-03-29 16:59:08 UTC 
I make sure to set show_save_dialog to TRUE under interactive mode.
Comment 7 Cosimo Cecchi 2012-05-08 16:24:52 UTC 
Comment on attachment 210788 [details] [review]
Update .gitignore

Attachment 210788 [details] pushed as c70c400 - Update .gitignore
Comment 8 Cosimo Cecchi 2012-05-08 16:25:58 UTC 
We agreed a GSetting is not the right approach - a command line switch would probably be better.

Closing this as a duplicate of 669629.

*** This bug has been marked as a duplicate of bug 669629 ***