D11253: Improve discoverability for changing the wallpaper

2018-03-27 Thread Nathaniel Graham
ngraham updated this revision to Diff 30755. ngraham added a comment. Give up on the unpopular text changes and only change the icon REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D11253?vs=30327=30755 BRANCH arcpatch-D11253 REVISION DETAIL

D11253: Improve discoverability for changing the wallpaper

2018-03-27 Thread Marco Martin
mart added a comment. I would prefer the new icon, but still "Configure Desktop" for brevity REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D11253 To: ngraham, #plasma, davidedmundson Cc: mart, hein, broulik, richardbowen, plasma-devel, ragreen, ZrenBot,

D11253: Improve discoverability for changing the wallpaper

2018-03-26 Thread Richard Bowen
richardbowen added a comment. The preferences-desktop-wallpaper icon beside "Configure Desktop" works. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D11253 To: ngraham, #plasma, davidedmundson Cc: mart, hein, broulik, richardbowen, plasma-devel, ragreen,

D11253: Improve discoverability for changing the wallpaper

2018-03-26 Thread Nathaniel Graham
ngraham added a comment. Is everyone still ambivalent about this? Is there anything I can do to make it more palatable, or should I give up? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D11253 To: ngraham, #plasma, davidedmundson Cc: mart, hein, broulik,

D11253: Improve discoverability for changing the wallpaper

2018-03-23 Thread Richard Bowen
richardbowen added a comment. In D11253#232632 , @ngraham wrote: > In D11253#232607 , @richardbowen wrote: > > > Since wallpaper is a feature of the desktop I don't think mentioning wallpaper in

D11253: Improve discoverability for changing the wallpaper

2018-03-23 Thread Nathaniel Graham
ngraham added a comment. In D11253#232607 , @richardbowen wrote: > Since wallpaper is a feature of the desktop I don't think mentioning wallpaper in the context menu title is necessary. The entire point of this patch is my assertion

D11253: Improve discoverability for changing the wallpaper

2018-03-23 Thread Richard Bowen
richardbowen added a comment. Since wallpaper is a feature of the desktop I don't mentioning wallpaper in the context menu title is necessary. "Configure Desktop" sounds a little technical but with the wallpaper image beside it it should be more clear. the word Configure is used throught