Fixes for MDPI devices

Author: Rajat Talesra

utility/src/main/java/org/oppia/util/parser/HtmlParser.kt

@@ -7,6 +7,7 @@ import android.text.SpannableStringBuilder
 import android.text.Spanned
 import android.text.style.BulletSpan
 import android.widget.TextView
+import androidx.appcompat.app.AppCompatActivity
 import javax.inject.Inject
 
 private const val CUSTOM_IMG_TAG = "oppia-noninteractive-image"
@@ -17,6 +18,7 @@ private const val REPLACE_IMG_FILE_PATH_ATTRIBUTE = "src"
 /** Html Parser to parse custom Oppia tags with Android-compatible versions. */
 class HtmlParser private constructor(
   private val urlImageParserFactory: UrlImageParser.Factory,
+  private val activity: AppCompatActivity,
   private val entityType: String,
   private val entityId: String,
   private val imageCenterAlign: Boolean
@@ -62,7 +64,7 @@ class HtmlParser private constructor(
       val end = spannableBuilder.getSpanEnd(it)
       spannableBuilder.removeSpan(it)
       spannableBuilder.setSpan(
-        ImprovedBulletSpan(),
+        ImprovedBulletSpan(activity),
         start,
         end,
         Spanned.SPAN_INCLUSIVE_EXCLUSIVE
@@ -90,9 +92,9 @@ class HtmlParser private constructor(
     return spannable.delete(0, trimStart).delete(spannable.length - trimEnd, spannable.length)
   }
 
-  class Factory @Inject constructor(private val urlImageParserFactory: UrlImageParser.Factory) {
+  class Factory @Inject constructor(private val urlImageParserFactory: UrlImageParser.Factory, private val activity: AppCompatActivity) {
     fun create(entityType: String, entityId: String, imageCenterAlign: Boolean): HtmlParser {
-      return HtmlParser(urlImageParserFactory, entityType, entityId, imageCenterAlign)
+      return HtmlParser(urlImageParserFactory, activity, entityType, entityId, imageCenterAlign)
     }
   }
 }

utility/src/main/java/org/oppia/util/parser/ImprovedBulletSpan.kt

@@ -1,28 +1,29 @@
 package org.oppia.util.parser
 
+import android.content.Context
 import android.graphics.Canvas
 import android.graphics.Paint
 import android.graphics.Path
 import android.graphics.Path.Direction
 import android.text.Layout
 import android.text.Spanned
 import android.text.style.LeadingMarginSpan
+import androidx.appcompat.app.AppCompatActivity
+import org.oppia.util.R
 
 /**
  * Copy of [android.text.style.BulletSpan] from android SDK 28 with removed internal code.
  *
  * Reference: https://github.com/davidbilik/bullet-span-sample
  */
-class ImprovedBulletSpan(
-  private val bulletRadius: Int = STANDARD_BULLET_RADIUS,
-  private val gapWidth: Int = STANDARD_GAP_WIDTH,
-  val color: Int = STANDARD_COLOR
-) : LeadingMarginSpan {
+class ImprovedBulletSpan(activity: AppCompatActivity) : LeadingMarginSpan {
 
-  companion object {
-    private const val STANDARD_BULLET_RADIUS = 8
-    private const val STANDARD_GAP_WIDTH = 32
-    private const val STANDARD_COLOR = 0
+  private var bulletRadius: Int = 0
+  private var gapWidth: Int = 0
+
+  init {
+    bulletRadius = (activity as Context).resources.getDimensionPixelSize(R.dimen.bullet_radius)
+    gapWidth = (activity as Context).resources.getDimensionPixelSize(R.dimen.bullet_gap_width)
   }
 
   private var mBulletPath: Path? = null

utility/src/main/res/values/dimens.xml

@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources>
+  <dimen name="bullet_radius">4dp</dimen>
+  <dimen name="bullet_gap_width">16dp</dimen>
+</resources>