Merge branch 'main' into @ksienkiewicz/fix-list-styling

Author: hejsztynx

.github/workflows/docs-build.yml

@@ -39,3 +39,17 @@ jobs:
 
       - name: Build docs
         run: yarn build
+
+      - name: Check docs llms.txt
+        run: |
+          file=build/llms.txt
+          if [ ! -s "$file" ]; then
+            echo "::error::$file is missing or empty"
+            exit 1
+          fi
+          count=$(grep -cE '^- \[[^]]+\]\(https://docs\.swmansion\.com/react-native-enriched-html/[^)]+\)' "$file" || true)
+          if [ "$count" -eq 0 ]; then
+            echo "::error::$file lists no pages"
+            exit 1
+          fi
+          echo "llms.txt lists $count pages"

.maestro/enrichedInput/flows/preserve_typing_attributes_on_selection_change.yaml

@@ -0,0 +1,34 @@
+appId: swmansion.enriched.example
+---
+# Verifies that typing attributes are properly preserved on selection changes
+- launchApp
+
+- tapOn:
+    id: 'toggle-screen-button'
+
+- tapOn:
+    id: "editor-input"
+
+- tapOn:
+    id: "toolbar-bold"
+
+- inputText: 'bold text'
+- pressKey: Enter
+- pressKey: Enter
+- pressKey: Enter
+- inputText: 'another line'
+
+- doubleTapOn:
+    id: 'editor-input'
+    point: '20%, 75%'
+
+- tapOn:
+    id: 'editor-input'
+    point: '50%, 15%'
+
+- inputText: 'new'
+
+- runFlow:
+    file: '../subflows/capture_or_assert_screenshot.yaml'
+    env:
+      SCREENSHOT_NAME: 'preserve_typing_attributes_on_selection_changes'

.maestro/enrichedInput/screenshots/android/preserve_typing_attributes_on_selection_changes.png

@@ -31,6 +31,7 @@ import com.swmansion.enriched.common.updateOrderedListColumnMargins
 import com.swmansion.enriched.text.spans.EnrichedTextImageSpan
 import com.swmansion.enriched.text.spans.interfaces.EnrichedTextClickableSpan
 import com.swmansion.enriched.text.spans.interfaces.EnrichedTextSpan
+import com.swmansion.enriched.textinput.spans.EnrichedLineHeightSpan
 import kotlin.math.ceil
 
 class EnrichedTextView : AppCompatTextView {
@@ -42,13 +43,15 @@ class EnrichedTextView : AppCompatTextView {
   private var fontWeight: Int = ReactConstants.UNSET
   private var fontSize: Float = EnrichedConstants.TEXT_DEFAULT_FONT_SIZE
   private var fontSizeRaw: Float? = null
+  private var lineHeight: Float? = null
   private var htmlStyleMap: ReadableMap? = null
   var allowFontScaling: Boolean = EnrichedConstants.ALLOW_FONT_SCALING_DEFAULT
     set(value) {
       if (field == value) return
       field = value
       fontSizeRaw?.let { setFontSize(it) }
       htmlStyleMap?.let { setHtmlStyle(it) }
+      applyLineSpacing()
     }
 
   private var enrichedStyle: EnrichedTextStyle? = null
@@ -187,6 +190,7 @@ class EnrichedTextView : AppCompatTextView {
       parsedText = null
       this.text = text
     }
+    applyLineSpacing()
   }
 
   private fun parseText(
@@ -308,6 +312,33 @@ class EnrichedTextView : AppCompatTextView {
     setTextSize(TypedValue.COMPLEX_UNIT_PX, sizeInt)
   }
 
+  fun setLineHeight(height: Float) {
+    lineHeight = if (height <= 0f) null else height
+    applyLineSpacing()
+  }
+
+  private fun applyLineSpacing() {
+    val currentText = text ?: return
+    val spannable =
+      currentText as? Spannable ?: SpannableString(currentText)
+    spannable
+      .getSpans(0, spannable.length, EnrichedLineHeightSpan::class.java)
+      .forEach { spannable.removeSpan(it) }
+
+    lineHeight?.let {
+      spannable.setSpan(
+        EnrichedLineHeightSpan(it, allowFontScaling),
+        0,
+        spannable.length,
+        Spannable.SPAN_INCLUSIVE_INCLUSIVE,
+      )
+    }
+
+    if (spannable !== currentText) {
+      setText(spannable, BufferType.SPANNABLE)
+    }
+  }
+
   fun setFontFamily(family: String?) {
     if (family != fontFamily) {
       fontFamily = family

.maestro/enrichedInput/screenshots/ios/inline_styles_merge.png

@@ -53,6 +53,13 @@ class EnrichedTextViewManager :
     view?.setFontSize(value)
   }
 
+  override fun setLineHeight(
+    view: EnrichedTextView?,
+    value: Float,
+  ) {
+    view?.setLineHeight(value)
+  }
+
   override fun setFontFamily(
     view: EnrichedTextView?,
     value: String?,

.maestro/enrichedInput/screenshots/ios/inline_styles_removal.png

@@ -4,6 +4,8 @@ import android.content.Context
 import android.graphics.Typeface
 import android.graphics.text.LineBreaker
 import android.os.Build
+import android.text.Spannable
+import android.text.SpannableString
 import android.text.StaticLayout
 import android.text.TextPaint
 import android.text.TextUtils
@@ -21,6 +23,7 @@ import com.swmansion.enriched.common.GumboNormalizer
 import com.swmansion.enriched.common.allowFontScalingFromProps
 import com.swmansion.enriched.common.parser.EnrichedParser
 import com.swmansion.enriched.common.pixelFromSpOrDp
+import com.swmansion.enriched.textinput.spans.EnrichedLineHeightSpan
 import kotlin.math.ceil
 
 object MeasurementStore {
@@ -134,6 +137,13 @@ object MeasurementStore {
     return props.getBoolean("useHtmlNormalizer")
   }
 
+  private fun lineHeightFromProps(props: ReadableMap?): Float {
+    if (props == null || !props.hasKey("lineHeight") || props.isNull("lineHeight")) {
+      return 0f
+    }
+    return props.getDouble("lineHeight").toFloat()
+  }
+
   private fun getInitialFontSize(props: ReadableMap?): Float {
     val propsFontSize = props?.getDouble("fontSize")?.toFloat() ?: EnrichedConstants.TEXT_DEFAULT_FONT_SIZE
     val fontSize =
@@ -151,15 +161,31 @@ object MeasurementStore {
     props: ReadableMap?,
   ): Long {
     val fontSize = getInitialFontSize(props)
-    val text = getInitialText(context, fontSize.toInt(), props)
+    val rawText = getInitialText(context, fontSize.toInt(), props)
+    val lineHeight = lineHeightFromProps(props)
+    val allowFontScaling = allowFontScalingFromProps(props)
+
+    val measuredText: CharSequence =
+      if (lineHeight > 0f) {
+        val spannable = SpannableString(rawText)
+        spannable.setSpan(
+          EnrichedLineHeightSpan(lineHeight, allowFontScaling),
+          0,
+          spannable.length,
+          Spannable.SPAN_INCLUSIVE_INCLUSIVE,
+        )
+        spannable
+      } else {
+        rawText
+      }
 
     val fontFamily = props?.getString("fontFamily")
     val numberOfLines = props?.getInt("numberOfLines") ?: 0
     val ellipsizeMode = props?.getString("ellipsizeMode")
     val fontStyle = parseFontStyle(props?.getString("fontStyle"))
     val fontWeight = parseFontWeight(props?.getString("fontWeight"))
     val typeface = applyStyles(null, fontStyle, fontWeight, fontFamily, context.assets)
-    val size = measure(width, text, typeface, fontSize, numberOfLines, ellipsizeMode)
+    val size = measure(width, measuredText, typeface, fontSize, numberOfLines, ellipsizeMode)
 
     return size
   }

.maestro/enrichedInput/screenshots/ios/preserve_typing_attributes_on_selection_changes.png

@@ -572,7 +572,7 @@ class EnrichedTextInputView :
   }
 
   fun setLineHeight(height: Float) {
-    lineHeight = if (height == 0f) null else height
+    lineHeight = if (height <= 0f) null else height
     applyLineSpacing()
     layoutManager.invalidateLayout()
     forceScrollToSelection()