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 661998 - background: Add context menu
background: Add context menu
Status: RESOLVED WONTFIX
Product: gnome-settings-daemon
Classification: Core
Component: background
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
: 647845 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-10-17 13:26 UTC by Florian Müllner
Modified: 2012-03-09 18:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
background: Add context menu (5.88 KB, patch)
2011-10-17 13:26 UTC, Florian Müllner
reviewed Details | Review

Description Florian Müllner 2011-10-17 13:26:57 UTC
See patch.

I'm unsure whether the background should be considered "system space", in which case a shell menu would probably be a better choice. If a "normal" GTK+ menu is what we want, the attached patch is lacking support for triggering the menu by keyboard shortcuts.
Comment 1 Florian Müllner 2011-10-17 13:26:59 UTC
Created attachment 199210 [details] [review]
background: Add context menu

With desktop icons enabled, right clicks on the background trigger
a context menu, which among other icon-view specific items contains
a shortcut to the background settings panel.
Add a similar context menu when gsd is drawing the desktop.
Comment 2 Bastien Nocera 2011-10-17 13:36:23 UTC
Review of attachment 199210 [details] [review]:

Code looks correct. I'd be enclined to think that a shell menu would be more appropriate, but leaving that up to the designers.
Comment 3 Allan Day 2011-10-17 15:16:31 UTC
*** Bug 647845 has been marked as a duplicate of this bug. ***
Comment 4 Allan Day 2011-10-17 15:17:34 UTC
(In reply to comment #2)
...
> I'd be enclined to think that a shell menu would be more
> appropriate

Agreed.
Comment 5 Florian Müllner 2011-10-17 15:21:47 UTC
(In reply to comment #4)
> (In reply to comment #2)
> ...
> > I'd be enclined to think that a shell menu would be more
> > appropriate
> 
> Agreed.

Uhm - why did you close the shell bug then?
Comment 6 William Jon McCann 2011-10-17 15:22:59 UTC
I'm not sure this is a good idea. I think that interaction with the OS should be done via the top bar or the message tray. Encouraging the use of the background doesn't seem like a good idea. I don't think we'd want to offer any other options here anyway besides change wallpaper anyway. I think it is better to have a way to set the wallpaper directly from an image in Photos or explicitly in the System Settings - so that we can set both the lock screen background and wallpaper to be the same if the user desires.
Comment 7 Allan Day 2011-10-17 17:39:00 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #2)
> > ...
> > > I'd be enclined to think that a shell menu would be more
> > > appropriate
> > 
> > Agreed.
> 
> Uhm - why did you close the shell bug then?

Sorry, thought you wanted to track the bug from here...

(In reply to comment #6)
> I'm not sure this is a good idea. I think that interaction with the OS should
> be done via the top bar or the message tray. Encouraging the use of the
> background doesn't seem like a good idea. I don't think we'd want to offer any
> other options here anyway besides change wallpaper anyway. I think it is better
> to have a way to set the wallpaper directly from an image in Photos or
> explicitly in the System Settings - so that we can set both the lock screen
> background and wallpaper to be the same if the user desires.

I think it is logical to enable someone to change the background by directly clicking on it. Also, changing the background using a context menu is a convention that a lot of people are familiar with.

The menu could contain a single item ('Change Wallpaper...') that would open the appropriate systems settings panel.
Comment 8 William Jon McCann 2011-10-17 18:03:03 UTC
I'll reiterate that I think this isn't a good idea. I don't think having a single item in a menu is a good reason to have the menu. I don't think we want to have the equivalent option on the lock screen or in the overview and generally speaking we don't expect people to be looking at empty backgrounds.

Add new pathways like this comes at a cost.
Comment 9 Bastien Nocera 2012-03-09 18:27:31 UTC
Agreed with Jon, closing.