Review Request 118290: Streamline Comment fields of KCMs

[David Edmundson]

> On May 26, 2014, 10:38 a.m., Christoph Feck wrote:
> > kcms/colors/colors.desktop, line 106
> > <https://git.reviewboard.kde.org/r/118290/diff/1/?file=274663#file274663line106>
> >
> >     I am not native english, but I think "Application's Color Scheme" is correct.

Application's color scheme = the application owns the colour scheme

Application Color Scheme = color scheme for applications.


If this was for configuring the colour scheme from within an application that affected only that app, your comment would make sense. 
For the case of the KCM I think "Application Color Scheme" works better.


> On May 26, 2014, 10:38 a.m., Christoph Feck wrote:
> > kcms/dateandtime/clock.desktop, line 107
> > <https://git.reviewboard.kde.org/r/118290/diff/1/?file=274665#file274665line107>
> >
> >     Is there a guideline for using the '&' character vs. "and"? I suggest to always use the word (which you also use e.g. for "Login and Logout") to make it consistent.

There's a guideline on no abbreviations. I think & counts as one.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118290/#review58456
-----------------------------------------------------------


On May 26, 2014, 8:49 a.m., Sebastian Kügler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118290/
> -----------------------------------------------------------
> 
> (Updated May 26, 2014, 8:49 a.m.)
> 
> 
> Review request for Plasma and KDE Usability.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Streamline Comment fields of KCMs
> 
> This patch applies a common language and type-setting to the
> systemsettings modules in kde-workspace.
> 
> Considerations:
> - The comment field might repeat the name, or give more detail about the
>   specific settings on this page, this makes sense with how
>   systemsettings and kcmshell present it
> - Mentioning the words settings, configure, options, etc. is avoided --
>   it is clear from the context that these are settings and options.
> - Title-case throughout in line with human interface guidelines, see
>   http://techbase.kde.org/Projects/Usability/HIG/Capitalization
> - The comment ends up being the title, so the
> - tech slang is avoided as much as possible, but left in where really
>   necessary (hi Phonon), there were a few mentions of ~"KDE Settings",
>   which don't make sense in a properly branded universe.
> - I've left the Name field "mostly untouched", as that one is key for
>   the user to find the right module in systemsettings' icon view and in
>   the sidebars
> 
> The same treatment needs to be done for a bunch of other KCMs that we
> end up installing from other repos. This is a good start, however.
> 
> 
> Diffs
> -----
> 
>   kcms/autostart/autostart.desktop 5a1048e72415d083040c970320fdca3b8d059f20 
>   kcms/access/kcmaccess.desktop f859504cf914ba79841ccf39f9e28dba5a4255d3 
>   kcms/bell/bell.desktop 906aff736f5a37c0c82d8a947d2938f3251ab132 
>   kcms/colors/colors.desktop 409f665a30faee826dbe1d70dab7f2fa3ec46bd3 
>   kcms/componentchooser/componentchooser.desktop 214c4f1a879dd0d7cd39c92b4510cdd1173f737c 
>   kcms/dateandtime/clock.desktop acaca1a3423f427e6cbb930e0f1e45ca82ef5c03 
>   kcms/desktoppaths/desktoppath.desktop da351d95e254cd5ae13c266f77213ad51e96fe0a 
>   kcms/desktoptheme/desktoptheme.desktop 602f04cfd5b2cb61d561a5961ddbe517f70995c6 
>   kcms/emoticons/emoticons.desktop bbbd24a8a62e871ef1df9930f344a362f33cae58 
>   kcms/fonts/fonts.desktop 99a88c4d6ca8526dd565733dba203536ddc63589 
>   kcms/formats/formats.desktop d29d893e2fcfbc8062d789aea9c74416c4a78b37 
>   kcms/hardware/display/display.desktop 0d8b7e667637f05a34c4b573d7958e90436ec792 
>   kcms/hardware/joystick/joystick.desktop e5ff24a8b95472e926d0ae91399c0c5f5828bdeb 
>   kcms/icons/icons.desktop 1dfcf8cda7fb04f42597071070787b20b166ecb3 
>   kcms/input/cursortheme.desktop 4a5feed001082dd2dac1c574c4d584bd70e45bd4 
>   kcms/input/mouse.desktop 5df856e30f8ed7e0e39338fe12da0ac54a52f6ac 
>   kcms/kded/kcmkded.desktop 76e299dfc5f3d223577bd62c91616f0b8988f979 
>   kcms/keyboard/kcm_keyboard.desktop 01f95420e295e26520bf2eda88fbaf02c3f09a62 
>   kcms/keys/keys.desktop 30a4bc8c0fea4767bd217671f8d624d563593552 
>   kcms/kfontinst/kcmfontinst/fontinst.desktop 6b1372545d5cabb560930c3dec5c9f3d1873e418 
>   kcms/knotify/kcmnotify.desktop cd25cd4dbfa2a2759eaec554c55e45fd133995ec 
>   kcms/ksmserver/kcmsmserver.desktop c461be78642e27b782f0be65d676244ca2bde80a 
>   kcms/ksplash/ksplashthememgr.desktop 711fbc335a4ccbf6e099e129302f40ed1d2929e9 
>   kcms/launch/kcmlaunch.desktop fbb3ca602f558075060f8e007ce000b4b78f8834 
>   kcms/phonon/kcm_phonon.desktop 46eecfffc93baecb7b77c0dcef32e9222abebb07 
>   kcms/spellchecking/spellchecking.desktop 92a76cfe63336a34f8f29b44b0e2ae551139a33c 
>   kcms/standard_actions/standard_actions.desktop 54a3ea84fe40dea252d1925d9f07cf6f2882c848 
>   kcms/style/style.desktop fd19bc4b204ca102081629fd6132b14857c4e4bc 
>   kcms/workspaceoptions/workspaceoptions.desktop 3e318660b0258c17bc7641efb1d7f1e84a1eed44 
> 
> Diff: https://git.reviewboard.kde.org/r/118290/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sebastian Kügler
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140526/10979e5a/attachment.html>