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

Authored by ngraham on Sep 24 2018, 11:05 PM.

Details

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 add any information
  • "Hinting style" -> "Hinting" -> same thing; "style" is just filler

Also added context to help translators come up with the best translations for the now shorter strings.

Test Plan

Diff Detail

Repository
R119 Plasma Desktop
Branch
remove-filler-words-from-fonts-kcm (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3190
Build 3208: arc lint + arc unit
ngraham created this revision.Sep 24 2018, 11:05 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 24 2018, 11:05 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Sep 24 2018, 11:05 PM
ngraham edited the summary of this revision. (Show Details)Sep 24 2018, 11:06 PM
ngraham edited the test plan for this revision. (Show Details)

Looks good to me +1

mart accepted this revision.Sep 28 2018, 8:50 AM
This revision is now accepted and ready to land.Sep 28 2018, 8:50 AM
ngraham closed this revision.Sep 28 2018, 5:35 PM