diff options
author | Raph Levien <raph@google.com> | 2014-06-12 09:19:03 -0700 |
---|---|---|
committer | Raph Levien <raph@google.com> | 2014-06-12 09:23:21 -0700 |
commit | 9f9f3b1ef40f7358dca6acd9dfef686cedefb6aa (patch) | |
tree | 5b1d74430ed33db3353faa9ae6ea730e779b2e77 | |
parent | 1c7b403ad8026fdebd0185319183d0a7a2c377bd (diff) | |
download | android_frameworks_minikin-9f9f3b1ef40f7358dca6acd9dfef686cedefb6aa.tar.gz android_frameworks_minikin-9f9f3b1ef40f7358dca6acd9dfef686cedefb6aa.tar.bz2 android_frameworks_minikin-9f9f3b1ef40f7358dca6acd9dfef686cedefb6aa.zip |
Tighten requirements for fake bold
The simple predicate for fake bold (2 or more grades darker than
requested) was applying it to thin (100 weight) when normal was
requested. This patch tightens the predicate to also require that
the requested weight be in the bold range.
Fix for bug 15588352 "sans-serif-thin doesn't work on lockscreen"
Change-Id: Id9988bd149a9c8a7c943e3b221f7fb4b37fb6ddb
-rw-r--r-- | libs/minikin/FontFamily.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/libs/minikin/FontFamily.cpp b/libs/minikin/FontFamily.cpp index 9106f63..ad8120f 100644 --- a/libs/minikin/FontFamily.cpp +++ b/libs/minikin/FontFamily.cpp @@ -119,9 +119,11 @@ static int computeMatch(FontStyle style1, FontStyle style2) { } static FontFakery computeFakery(FontStyle wanted, FontStyle actual) { - // If desired weight is 2 or more grades higher than actual - // (for example, medium 500 -> bold 700), then select fake bold. - bool isFakeBold = (wanted.getWeight() - actual.getWeight()) >= 2; + // If desired weight is bold or darker, and 2 or more grades higher + // than actual (for example, medium 500 -> bold 700), then select + // fake bold. + int wantedWeight = wanted.getWeight(); + bool isFakeBold = wantedWeight >= 7 && (wantedWeight - actual.getWeight()) >= 2; bool isFakeItalic = wanted.getItalic() && !actual.getItalic(); return FontFakery(isFakeBold, isFakeItalic); } |