[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels
From: Nathaniel Graham <noreply () phabricator ! kde ! org>
Date: 2018-09-24 23:05:24
Message-ID: c68027a9a946b97261598aeec901efd9 () localhost ! localdomain
[Download RAW message or body]
[Attachment #2 (text/plain)]
ngraham created this revision.
ngraham added reviewers: VDG, Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
ngraham requested review of this revision.
REVISION SUMMARY
This is a new version of D13593 <https://phabricator.kde.org/D13593> with only the \
non-controversial changes (plus one new change, because I just can't resist stirring \
the pot :) ):
- "Use Anti-Aliasing" -> "Anti-Aliasing": The verb is not needed and in fact \
becomes awkward with the combobox's options
- "Sub-pixel rendering type" -> "Sub-pixel rendering"; "type" is just useless \
filler that lengthens the string and doesn't actually add any \
information
- "Hinting style" -> "Hinting" -> same thing; "style" is just useless filler
Also added context to help translators come up with the best translations for the \
now shorter strings
TEST PLAN
[image goes here]
REPOSITORY
R119 Plasma Desktop
BRANCH
remove-filler-words-from-fonts-kcm (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D15738
AFFECTED FILES
kcms/fonts/package/contents/ui/main.qml
To: ngraham, #vdg, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, \
abetts, sebas, apol, mart
[Attachment #3 (text/html)]
<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: \
VDG, Plasma.<br />Herald added a project: Plasma.<br />Herald added a subscriber: \
plasma-devel.<br />ngraham requested review of this revision. </td><a \
style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; \
color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; \
background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; \
border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15738">View \
Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is \
a new version of <a href="https://phabricator.kde.org/D13593" \
style="background-color: #e7e7e7; border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D13593</a> with only the \
non-controversial changes (plus one new change, because I just can't resist \
stirring the pot :) ):</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">"Use Anti-Aliasing" -> \
"Anti-Aliasing": The verb is not needed and in fact becomes awkward with \
the combobox's options</li> <li class="remarkup-list-item">"Sub-pixel \
rendering type" -> "Sub-pixel rendering"; "type" is just \
useless filler that lengthens the string and doesn't actually add any \
information</li> <li class="remarkup-list-item">"Hinting style" -> \
"Hinting" -> same thing; "style" is just useless filler</li> \
</ul>
<p>Also added context to help translators come up with the best translations for the \
now shorter strings</p></div></div><br /><div><strong>TEST \
PLAN</strong><div><p>[image goes here]</p></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>remove-filler-words-from-fonts-kcm (branched \
from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D15738">https://phabricator.kde.org/D15738</a></div></div><br \
/><div><strong>AFFECTED \
FILES</strong><div><div>kcms/fonts/package/contents/ui/main.qml</div></div></div><br \
/><div><strong>To: </strong>ngraham, VDG, Plasma<br /><strong>Cc: \
</strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, \
jensreuterberg, abetts, sebas, apol, mart<br /></div>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic