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 785364 - Add toggle for disabling thumbnailers
Add toggle for disabling thumbnailers
Status: RESOLVED OBSOLETE
Product: gnome-control-center
Classification: Core
Component: Privacy
unspecified
Other All
: Normal enhancement
: ---
Assigned To: Rui Matos
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-07-24 18:15 UTC by Bastien Nocera
Modified: 2021-06-09 16:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
privacy: Add a toggle for all thumbnailers (3.69 KB, patch)
2017-07-25 08:16 UTC, Bastien Nocera
none Details | Review
privacy: Add a toggle for all thumbnailers (3.66 KB, patch)
2017-07-25 08:18 UTC, Bastien Nocera
none Details | Review
thumbnailers disable (16.75 KB, image/png)
2017-07-25 08:18 UTC, Bastien Nocera
  Details
privacy: Add a toggle for all thumbnailers (3.90 KB, patch)
2017-07-25 20:53 UTC, Bastien Nocera
rejected Details | Review

Description Bastien Nocera 2017-07-24 18:15:23 UTC
gnome-desktop follows a setting in gsettings-desktop-schemas to disable running thumbnailers. We should have a GUI control to turn it off.
Comment 1 Bastien Nocera 2017-07-25 08:16:43 UTC
Created attachment 356340 [details] [review]
privacy: Add a toggle for all thumbnailers
Comment 2 Bastien Nocera 2017-07-25 08:18:01 UTC
Created attachment 356341 [details] [review]
privacy: Add a toggle for all thumbnailers
Comment 3 Bastien Nocera 2017-07-25 08:18:40 UTC
Created attachment 356342 [details]
thumbnailers disable
Comment 4 Rui Matos 2017-07-25 14:07:02 UTC
Review of attachment 356341 [details] [review]:

otherwise looks fine, if UI has been reviewed feel free to push

::: panels/privacy/cc-privacy-panel.c
@@ +69,3 @@
   guint       abrt_watch_id;
 
+  GtkWidget  *thumbnailers_dialog;

unused

@@ +1412,3 @@
   self->priv->notification_settings = g_settings_new ("org.gnome.desktop.notifications");
   self->priv->location_settings = g_settings_new ("org.gnome.system.location");
+  self->priv->thumbnailers_settings = g_settings_new ("org.gnome.desktop.thumbnailers");

needs to be freed on finalize
Comment 5 Bastien Nocera 2017-07-25 20:53:31 UTC
Created attachment 356383 [details] [review]
privacy: Add a toggle for all thumbnailers
Comment 6 Rui Matos 2017-07-26 12:51:42 UTC
Review of attachment 356383 [details] [review]:

++
Comment 7 Bastien Nocera 2017-08-10 11:54:14 UTC
Review of attachment 356383 [details] [review]:

<aday[m]> hadess: the setting is going to break a fair amount of stuff; it would be better to find a way to keep some thumbnailing while mitigating the privacy implications. that said, if we do want it i'd probably want to integrate it into the existing usage/history/trash/temp files settings rather than growing a whole new "turn off thumbnails" thing - better to group these things logically rather than grow controls for technical details
Comment 8 André Klapper 2021-06-09 16:00:58 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new enhancement request ticket at
  https://gitlab.gnome.org/GNOME/gnome-control-center/-/issues/

Thank you for your understanding and your help.