lineSpacingAdd should not be negative, so I add a constraint.

hy2012 · Jul 18, 2016 · 3f16925 · 3f16925
1 parent 4860396
commit 3f16925
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/app/src/main/java/io/plaidapp/ui/widget/CollapsingTitleLayout.java b/app/src/main/java/io/plaidapp/ui/widget/CollapsingTitleLayout.java
@@ -235,7 +235,8 @@ private void recalculate(int width) {
         int fontHeight = Math.abs(fm.ascent - fm.descent) + fm.leading;
         final int baselineAlignedLineHeight =
                 (int) (fourDip * (float) Math.ceil(lineHeightHint / fourDip));
-        final int lineSpacingAdd = baselineAlignedLineHeight - fontHeight;
+        // Addition line spacing to match desired line height. Should be non-negative.
+        final int lineSpacingAdd = Math.max(0, baselineAlignedLineHeight - fontHeight);
 
         // now create the layout with our desired insets & line height
         createLayout(width, lineSpacingAdd);