react-native-advanced-text 0.2.4 → 0.2.7

package/android/src/main/java/com/advancedtext/AdvancedTextView.kt

@@ -25,7 +25,6 @@ import com.facebook.react.uimanager.events.RCTEventEmitter
 import android.text.Selection
 import android.graphics.Typeface
 import androidx.core.text.getSpans
-import com.facebook.react.common.assets.ReactFontManager
 
 class AdvancedTextView : TextView {
 
@@ -164,25 +163,9 @@ class AdvancedTextView : TextView {
     fun setAdvancedFontFamily(family: String) {
         if (fontFamily == family) return
         fontFamily = family
-        typeface = resolveTypeface(family, Typeface.NORMAL)
+        typeface = Typeface.create(family, Typeface.NORMAL)
     }
 
-    /**
-     * `Typeface.create(family, style)` only resolves OS-registered families; it
-     * silently substitutes a system font for anything bundled under
-     * `assets/fonts/` (the RN/Expo custom-font pipeline) instead of failing.
-     * The Fabric ShadowNode measures this component's text via
-     * `TextLayoutManager`, which resolves fonts through `ReactFontManager` -
-     * so a custom font here rendered with the substituted typeface has
-     * different glyph widths than what was measured, wrapping onto a
-     * different number of lines and producing extra height for some strings
-     * but not others. Going through `ReactFontManager` keeps the rendered
-     * typeface identical to the measured one (and is a no-op for system font
-     * names, which it falls back to `Typeface.create` for anyway).
-     */
-    private fun resolveTypeface(family: String, style: Int): Typeface =
-        ReactFontManager.getInstance().getTypeface(family, style, context.assets)
-
     fun setMenuOptions(menuOptions: List<String>) {
         if (this.menuOptions == menuOptions) return
         this.menuOptions = menuOptions
@@ -288,9 +271,9 @@ class AdvancedTextView : TextView {
         setTextSize(fontSize)
 
         typeface = when (fontWeight) {
-            "bold" -> resolveTypeface(fontFamily, Typeface.BOLD)
-            "italic" -> resolveTypeface(fontFamily, Typeface.ITALIC)
-            else -> resolveTypeface(fontFamily, Typeface.NORMAL)
+            "bold" -> Typeface.create(fontFamily, Typeface.BOLD)
+            "italic" -> Typeface.create(fontFamily, Typeface.ITALIC)
+            else -> Typeface.create(fontFamily, Typeface.NORMAL)
         }
 
 

package/common/cpp/react/renderer/components/AdvancedTextViewSpec/ShadowNodes.cpp

@@ -11,7 +11,6 @@
 
 #include <algorithm>
 #include <cctype>
-#include <limits>
 #include <string>
 
 #include <react/renderer/attributedstring/AttributedStringBox.h>
@@ -33,10 +32,6 @@ std::string toLowerAscii(std::string value) {
   return value;
 }
 
-// Fallback only: an approximate ratio of a single line's natural height to
-// its font size, used if a real measurement (Android) isn't available/valid.
-constexpr Float kLineHeightFudgeFactor = 1.2;
-
 } // namespace
 
 AdvancedTextViewShadowNode::AdvancedTextViewShadowNode(
@@ -47,47 +42,7 @@ AdvancedTextViewShadowNode::AdvancedTextViewShadowNode(
           static_cast<const AdvancedTextViewShadowNode&>(sourceShadowNode)
               .textLayoutManager_) {}
 
-#if defined(ANDROID)
-Float AdvancedTextViewShadowNode::measureNaturalSingleLineHeight(
-    const LayoutContext& layoutContext,
-    const TextAttributes& baseTextAttributes) const {
-  // Same font/size/weight/style as the real fragment, but with `lineHeight`
-  // cleared so this reports the font's own, unscaled single-line height --
-  // exactly what Android's `setLineSpacing(0, multiplier)` multiplies.
-  auto probeAttributes = baseTextAttributes;
-  probeAttributes.lineHeight = std::numeric_limits<Float>::quiet_NaN();
-
-  auto probe = AttributedString{};
-  probe.appendFragment(AttributedString::Fragment{
-      .string = "M",
-      .textAttributes = probeAttributes,
-      .parentShadowView = ShadowView(*this),
-  });
-
-  auto paragraphAttributes = ParagraphAttributes{};
-
-  auto textLayoutContext = TextLayoutContext{};
-  textLayoutContext.pointScaleFactor = layoutContext.pointScaleFactor;
-  textLayoutContext.surfaceId = getSurfaceId();
-
-  // Unconstrained so a single glyph can never be forced to wrap.
-  auto unconstrainedLayoutConstraints = LayoutConstraints{
-      {0, 0},
-      {std::numeric_limits<Float>::infinity(),
-       std::numeric_limits<Float>::infinity()}};
-
-  auto measurement = textLayoutManager_->measure(
-      AttributedStringBox{probe},
-      paragraphAttributes,
-      textLayoutContext,
-      unconstrainedLayoutConstraints);
-
-  return measurement.size.height;
-}
-#endif
-
-AttributedString AdvancedTextViewShadowNode::getAttributedString(
-    const LayoutContext& layoutContext) const {
+AttributedString AdvancedTextViewShadowNode::getAttributedString() const {
   const auto& props = getConcreteProps();
 
   auto textAttributes = TextAttributes::defaultTextAttributes();
@@ -113,32 +68,13 @@ AttributedString AdvancedTextViewShadowNode::getAttributedString(
     textAttributes.letterSpacing = props.letterSpacing;
   }
 
-  // The component treats `lineHeight` as a multiple of the font's own
-  // natural line height (see setLineSpacing(0, multiplier) in
-  // AdvancedTextView.kt / the NSParagraphStyle-based spacing in
-  // AdvancedTextView.mm) -- NOT the absolute per-line value
-  // `TextAttributes.lineHeight` expects. A multiplier of 1 *is* the natural
-  // line height, so there is nothing to override in that case (and
-  // overriding it with an approximation, as this used to do unconditionally,
-  // made even the "no lineHeight override" case measure taller or shorter
-  // than the real render).
-  if (props.lineHeight > 0 && props.lineHeight != 1.0) {
-#if defined(ANDROID)
-    // Android's natural single-line height depends on the resolved
-    // font/size/weight and isn't a fixed ratio of fontSize, so a constant
-    // approximation drifts from AdvancedTextView.kt's real
-    // setLineSpacing()-based layout by an amount that compounds with every
-    // wrapped line -- which is why the error scaled with paragraph length.
-    // Measure the real natural height for these exact attributes instead.
-    auto naturalLineHeight =
-        measureNaturalSingleLineHeight(layoutContext, textAttributes);
-    textAttributes.lineHeight = naturalLineHeight > 0
-        ? props.lineHeight * naturalLineHeight
-        : props.lineHeight * textAttributes.fontSize * kLineHeightFudgeFactor;
-#else
+  if (props.lineHeight > 0) {
+    // The component treats `lineHeight` as a multiple of the font size
+    // (see AdvancedTextView.mm / AdvancedTextView.kt). `TextAttributes`
+    // expects an absolute value; approximate the platform default single
+    // line height as 1.2 * fontSize.
     textAttributes.lineHeight =
-        props.lineHeight * textAttributes.fontSize * kLineHeightFudgeFactor;
-#endif
+        props.lineHeight * textAttributes.fontSize * 1.2;
   }
 
   auto attributedString = AttributedString{};
@@ -173,7 +109,7 @@ Size AdvancedTextViewShadowNode::measureContent(
   textLayoutContext.surfaceId = getSurfaceId();
 
   auto measurement = textLayoutManager_->measure(
-      AttributedStringBox{getAttributedString(layoutContext)},
+      AttributedStringBox{getAttributedString()},
       paragraphAttributes,
       textLayoutContext,
       layoutConstraints);

package/common/cpp/react/renderer/components/AdvancedTextViewSpec/ShadowNodes.h

@@ -86,20 +86,7 @@ class AdvancedTextViewShadowNode final : public ConcreteViewShadowNode<
    * Builds a single-fragment `AttributedString` from the component props
    * (text + font attributes), mirroring how the native views render it.
    */
-  AttributedString getAttributedString(const LayoutContext& layoutContext) const;
-
-#if defined(ANDROID)
-  /*
-   * Measures the natural (multiplier == 1) height of a single line set in
-   * `baseTextAttributes`, by measuring a single glyph with `lineHeight`
-   * cleared. Used to turn the component's `lineHeight` multiplier prop into
-   * the absolute value `TextAttributes.lineHeight` expects, using this
-   * font's *real* metrics instead of an approximation.
-   */
-  Float measureNaturalSingleLineHeight(
-      const LayoutContext& layoutContext,
-      const TextAttributes& baseTextAttributes) const;
-#endif
+  AttributedString getAttributedString() const;
 
   // Lazily created on first measure, then reused / copied across clones.
   mutable std::shared_ptr<const TextLayoutManager> textLayoutManager_;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-advanced-text",
-  "version": "0.2.4",
+  "version": "0.2.7",
   "description": " Advanced text component for React Native with custom select options.",
   "main": "./lib/module/index.js",
   "types": "./lib/typescript/src/index.d.ts",