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 674640 - Remove screenshot dialogue
Remove screenshot dialogue
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Movie player
unspecified
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-04-23 16:47 UTC by Bastien Nocera
Modified: 2012-06-28 16:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot: Add filename builder helpers (10.90 KB, patch)
2012-06-28 11:52 UTC, Bastien Nocera
committed Details | Review
screenshot: Add license exception (1.54 KB, patch)
2012-06-28 11:52 UTC, Bastien Nocera
committed Details | Review
screenshot: Remove UI and flash the video instead (41.53 KB, patch)
2012-06-28 11:52 UTC, Bastien Nocera
committed Details | Review

Description Bastien Nocera 2012-04-23 16:47:29 UTC
And instead use the new facilities in gnome-shell to make it blink like an ace.

See:
https://bugzilla.gnome.org/show_bug.cgi?id=652952
and:
https://bugzilla.gnome.org/show_bug.cgi?id=668618
Comment 1 Bastien Nocera 2012-06-28 11:52:10 UTC
Created attachment 217514 [details] [review]
screenshot: Add filename builder helpers

From gnome-screenshot
Comment 2 Bastien Nocera 2012-06-28 11:52:16 UTC
Created attachment 217515 [details] [review]
screenshot: Add license exception
Comment 3 Bastien Nocera 2012-06-28 11:52:23 UTC
Created attachment 217516 [details] [review]
screenshot: Remove UI and flash the video instead

This matches the UI used in the shell where the captured
section of the screen flashes, and the video is automatically
saved in the user's Pictures directory.
Comment 4 Bastien Nocera 2012-06-28 11:54:40 UTC
Cosimo, could you check whether it's alright to add the license exception to this gnome-screenshot code? Are you the sole copyright holder for that portion of code?
Comment 5 Cosimo Cecchi 2012-06-28 15:54:17 UTC
(In reply to comment #4)
> Cosimo, could you check whether it's alright to add the license exception to
> this gnome-screenshot code? Are you the sole copyright holder for that portion
> of code?

I originally wrote it, and as far as I can see from the git history, it only has been touched for minor modifications by Jon, Matthias and Ray. I think it's alright to add the license exception.
Comment 6 Bastien Nocera 2012-06-28 16:26:29 UTC
Attachment 217514 [details] pushed as b68db23 - screenshot: Add filename builder helpers
Attachment 217515 [details] pushed as bb598ec - screenshot: Add license exception
Attachment 217516 [details] pushed as 1249d1e - screenshot: Remove UI and flash the video instead