D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels

2018-09-28 Thread Nathaniel Graham 
ngraham closed this revision.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D15738

To: ngraham, #vdg, #plasma, mart
Cc: abetts, svenmauch, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, apol, mart

D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels

2018-09-25 Thread Andres Betts 
abetts added a comment.


  +1

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D15738

To: ngraham, #vdg, #plasma
Cc: abetts, svenmauch, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, sebas, apol, mart

D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels

2018-09-25 Thread Sven Mauch 
svenmauch added a comment.


  Looks good to me +1

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D15738

To: ngraham, #vdg, #plasma
Cc: svenmauch, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

D15738: [Fonts KCM] remove filler words from anti-aliasing settings' labels

2018-09-24 Thread Nathaniel Graham 
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  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