D11757: Polish sidebar header appearance

2018-03-28 Thread Richard Bowen
richardbowen added a comment. ok, cool. REPOSITORY R124 System Settings REVISION DETAIL https://phabricator.kde.org/D11757 To: ngraham, mart, #plasma, #vdg Cc: davidedmundson, abetts, richardbowen, plasma-devel, ragreen, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol,

D11757: Polish sidebar header appearance

2018-03-28 Thread Richard Bowen
richardbowen added a comment. The greyborder around the submenu heading isn't consistent with the other main menu heading which have no border, and inconsistent with the Kirigami back button style but otherwise, it looks good. REPOSITORY R124 System Settings REVISION DETAIL

D11757: Polish sidebar header appearance

2018-03-27 Thread Richard Bowen
richardbowen added a comment. The whole area of the Kirigami style back button is clickable. REPOSITORY R124 System Settings REVISION DETAIL https://phabricator.kde.org/D11757 To: ngraham, mart, #plasma, #vdg Cc: richardbowen, plasma-devel, ragreen, ZrenBot, lesliezhai, ali-mohamed,

D11757: Polish sidebar header appearance

2018-03-27 Thread Richard Bowen
richardbowen added a comment. Yea, the hover border isn't necessary. Maybe the back navigation should work like how it does with Kirigami. REPOSITORY R124 System Settings REVISION DETAIL https://phabricator.kde.org/D11757 To: ngraham, mart, #plasma, #vdg Cc: richardbowen, plasma-devel,

D11757: Polish sidebar header appearance

2018-03-27 Thread Richard Bowen
richardbowen added a comment. I prefer the system settings as it currently is in 5.12, though the submenu heading could be a little bit darker. As for the changes made, I think the person that made the main sidebar header color different from the sub-sidebar header color did so

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-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 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

D11253: Add the word "Wallpaper" into the context and Toolbox menus to help users figure out how to change the wallpaper

2018-03-23 Thread Richard Bowen
richardbowen added a comment. Also how about a preferences shortcut for "Desktop Wallpaper" (or some other suitable text) searchable from the start menu and krunner? This would be similar to the other desktop preferences that are searchable like "Desktop Theme", "Desktop Effects", etc..

D11253: Add the word "Wallpaper" into the context and Toolbox menus to help users figure out how to change the wallpaper

2018-03-23 Thread Richard Bowen
richardbowen added a comment. In D11253#232189 , @mart wrote: > what about as text using Configure-desktop and as icon preferences-desktop-wallpaper?F5763655: Spectacle.T27450.png +1 A big

D11308: Use the default Plasma wallpaper on the lock screen

2018-03-14 Thread Richard Bowen
richardbowen added a comment. Blue screen is fine with me but hey, I like blue. REPOSITORY R133 KScreenLocker REVISION DETAIL https://phabricator.kde.org/D11308 To: ngraham, #plasma, #vdg, graesslin Cc: richardbowen, zzag, Pitel, progwolff, abetts, hein, mart, graesslin, plasma-devel,

D11253: Add the word "Wallpaper" into the context and Toolbox menus to help users figure out how to change the wallpaper

2018-03-11 Thread Richard Bowen
richardbowen added a comment. I wonder if one should dumb down the interface to accommodate this word scanning habit, where one doesn't really read stuff one just scans the screen for a keyword and then gets frustrated when the keyword (like "Wallpaper") doesn't show up . Well if it works

D11253: Add the word "Wallpaper" into the context and Toolbox menus to help users figure out how to change the wallpaper

2018-03-11 Thread Richard Bowen
richardbowen added a comment. Windows has the word "Personalize" which is quite descriptive. "Customize Desktop" sounds fine as well. but I think the word "Configure" is used more often throughtout the Plasma desktop. How often do you see users complaining about not knowing how to set