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 656977 - If icons aren't shown on the Desktop, disable the options to show specific icons
If icons aren't shown on the Desktop, disable the options to show specific icons
Status: RESOLVED FIXED
Product: gnome-tweak-tool
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Tweak Tool maintainer(s)
GNOME Tweak Tool maintainer(s)
: 700026 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-08-20 19:08 UTC by Jeremy Bicha
Modified: 2013-07-17 21:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Also made a Pull Request for the same patch, Please Review it. (2.51 KB, patch)
2013-04-18 20:41 UTC, Vishrut Mehta
none Details | Review
Patch to make the dependent options insensitive when show-desktop-icons is off. (855 bytes, patch)
2013-04-22 19:08 UTC, Fotis Tsamis
none Details | Review
desktop options are only sensitive if filemanager manages the desktop (2.81 KB, patch)
2013-07-17 21:12 UTC, John Stowers
committed Details | Review

Description Jeremy Bicha 2011-08-20 19:08:16 UTC
If "Have file manager handle the desktop" is off, then the other options on the Desktop (File Manager) panel won't have any visible effect if they are changed. They should therefore be disabled as long as that setting is off, to make it clear to users that the first settings affects all of the other desktop icon settings.
Comment 1 Vishrut Mehta 2013-04-17 10:55:14 UTC
So,
Do we need to make the enabled icons like Trash, Computer, etc. visible on the scrren if they are enabled and the "Have file manager handle the desktop" is off ???
OR
User will not be allowed to enable the other option, if "Have file manager handle the desktop" is off ??
Comment 2 Vishrut Mehta 2013-04-17 11:08:56 UTC
So,
Do we need to make the enabled icons like Trash, Computer, etc. visible on the scrren if they are enabled and the "Have file manager handle the desktop" is off ???
OR
User will not be allowed to enable the other option, if "Have file manager handle the desktop" is off ??
Comment 3 Matthias Clasen 2013-04-17 22:38:06 UTC
I think all the other 'Show...on desktop' options should be disabled if the file manager switch is off
Comment 4 John Stowers 2013-04-18 10:20:08 UTC
(thanks Mattias, I agree. This is my "explain how you might fix this bug and why" test for GSOC students)
Comment 5 Vishrut Mehta 2013-04-18 17:19:07 UTC
I have fixed this bug, but need reviwing. The Pull Request's link:
https://github.com/nzjrs/gnome-tweak-tool/pull/2

It does the following rask:
-> When "Have file manager handle the desktop" is off from start, then all the other options will also be OFF by default.
-> If this option is on, then the other options will behave normally, but when in between this option is OFF, then all the other options are disabled (ie. are set to OFF)
Comment 6 Vishrut Mehta 2013-04-18 20:41:37 UTC
Created attachment 241861 [details] [review]
Also made a Pull Request for the same patch, Please Review it.
Comment 7 Fotis Tsamis 2013-04-22 19:08:54 UTC
Created attachment 242160 [details] [review]
Patch to make the dependent options insensitive when show-desktop-icons is off.

I think you need to retain the status of the dependent options (on/off) and make them insensitive when show-desktop-icons is off, instead of turning them all off. Also, I find it more appropriate to make this change inside the tweak_nautilus.py file.

I've attached a patch with my approach.
Comment 8 Jeremy Bicha 2013-05-20 16:26:02 UTC
*** Bug 700026 has been marked as a duplicate of this bug. ***
Comment 9 John Stowers 2013-07-17 21:12:44 UTC
Created attachment 249447 [details] [review]
desktop options are only sensitive if filemanager manages the desktop
Comment 10 John Stowers 2013-07-17 21:13:11 UTC
Comment on attachment 249447 [details] [review]
desktop options are only sensitive if filemanager manages the desktop

Attachment 249447 [details] pushed as a46bbdb - desktop options are only sensitive if filemanager manages the desktop
Comment 11 John Stowers 2013-07-17 21:25:28 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.