[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-i18n-doc
Subject: Re: Review Request 118338: [kwin] Streamline Comment fields of KCMs
From: Sebastian_Kügler <sebas () kde ! org>
Date: 2014-05-27 15:54:28
Message-ID: 20140527155428.21300.76648 () probe ! kde ! org
[Download RAW message or body]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118338/
-----------------------------------------------------------
(Updated May 27, 2014, 3:54 p.m.)
Review request for kwin, Localization and Translation (l10n) and KDE Usability.
Changes
-------
Addressed comments (as far as I agree with them).
No "Graphics", in compositor, anyway.
Repository: kwin
Description
-------
[kwin] Streamline Comment fields of KCMs
This patch applies a common language and type-setting to the
systemsettings modules in kwin.
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
- 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 most prominent change is "Compositor" to "Graphics Compositor" (in
the Title field). This is more like a "suggestion", if you don't like
it, I'll take it out again. Reasoning: "Compositor" is a very technical
term, people who know graphics might recognize it. Others might not
understand that "composite" is about graphics -- hence the
specialization.
Diffs (updated)
-----
kcmkwin/kwinoptions/kwinfocus.desktop d50fb82
kcmkwin/kwinoptions/kwinmoving.desktop cd6dcbc
kcmkwin/kwinoptions/kwinoptions.desktop 6075a5c
kcmkwin/kwinrules/kwinrules.desktop cded437
kcmkwin/kwinscreenedges/kwinscreenedges.desktop 404f28a
kcmkwin/kwintabbox/kwintabbox.desktop 5955e10
kcmkwin/kwinoptions/kwinadvanced.desktop 6cbb95a
kcmkwin/kwindesktop/desktop.desktop 9e9506c
kcmkwin/kwinoptions/kwinactions.desktop 0ebd40c
kcmkwin/kwindecoration/kwindecoration.desktop 51a2034
kcmkwin/kwincompositing/kwincompositing.desktop 087b589
kcmkwin/kwincompositing/kcmkwineffects.desktop 390199b
Diff: https://git.reviewboard.kde.org/r/118338/diff/
Testing
-------
Thanks,
Sebastian Kügler
[Attachment #3 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/118338/">https://git.reviewboard.kde.org/r/118338/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for kwin, Localization and Translation (l10n) and KDE \
Usability.</div> <div>By Sebastian Kügler.</div>
<p style="color: grey;"><i>Updated May 27, 2014, 3:54 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Addressed comments (as far as I agree with them).
No "Graphics", in compositor, anyway.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kwin
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">[kwin] Streamline Comment fields of KCMs
This patch applies a common language and type-setting to the
systemsettings modules in kwin.
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
- 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 most prominent change is "Compositor" to "Graphics \
Compositor" (in the Title field). This is more like a "suggestion", if \
you don't like it, I'll take it out again. Reasoning: "Compositor" \
is a very technical term, people who know graphics might recognize it. Others might \
not understand that "composite" is about graphics -- hence the
specialization.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> \
(updated)</h1> <ul style="margin-left: 3em; padding-left: 0;">
<li>kcmkwin/kwinoptions/kwinfocus.desktop <span style="color: \
grey">(d50fb82)</span></li>
<li>kcmkwin/kwinoptions/kwinmoving.desktop <span style="color: \
grey">(cd6dcbc)</span></li>
<li>kcmkwin/kwinoptions/kwinoptions.desktop <span style="color: \
grey">(6075a5c)</span></li>
<li>kcmkwin/kwinrules/kwinrules.desktop <span style="color: \
grey">(cded437)</span></li>
<li>kcmkwin/kwinscreenedges/kwinscreenedges.desktop <span style="color: \
grey">(404f28a)</span></li>
<li>kcmkwin/kwintabbox/kwintabbox.desktop <span style="color: \
grey">(5955e10)</span></li>
<li>kcmkwin/kwinoptions/kwinadvanced.desktop <span style="color: \
grey">(6cbb95a)</span></li>
<li>kcmkwin/kwindesktop/desktop.desktop <span style="color: \
grey">(9e9506c)</span></li>
<li>kcmkwin/kwinoptions/kwinactions.desktop <span style="color: \
grey">(0ebd40c)</span></li>
<li>kcmkwin/kwindecoration/kwindecoration.desktop <span style="color: \
grey">(51a2034)</span></li>
<li>kcmkwin/kwincompositing/kwincompositing.desktop <span style="color: \
grey">(087b589)</span></li>
<li>kcmkwin/kwincompositing/kcmkwineffects.desktop <span style="color: \
grey">(390199b)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118338/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic