fix(ios): selection reset (#42)

* fix(ios): collapse NitroText selection on outside taps
* refactor: streamline TextAncestorContext initialization in NitroText component

Author: patrickkabwe

cpp/NitroTextShadowNode.cpp

@@ -1,6 +1,6 @@
 //
 // NitroTextShadowNode.cpp
-// Moved implementations from header into translation unit
+//
 //
 
 #include "NitroTextShadowNode.hpp"

example/App.tsx

@@ -78,8 +78,10 @@ export default function App() {
           paragraph.
         </NitroText>
       </View>
+
+      {/* Rich Text Formatting using RN Text */}
       <View style={styles.section}>
-        <Text style={styles.sectionTitle}>Rich Text Formatting</Text>
+        <Text style={styles.sectionTitle}>Rich Text Formatting(RN Text)</Text>
         <Text style={styles.richText}>
           Welcome to the world of <Text style={styles.bold}>bold text</Text>,{' '}
           <Text style={styles.italic}>beautiful italics</Text>, and{' '}
@@ -112,8 +114,9 @@ export default function App() {
         </NitroText>
       </View>
 
+      {/* Layout Measurement using RN Text */}
       <View style={styles.section}>
-        <Text style={styles.sectionTitle}>Layout Measurement</Text>
+        <Text style={styles.sectionTitle}>Layout Measurement(RN Text)</Text>
         <Text style={styles.measuredText} onLayout={handleLayout}>
           This text demonstrates layout measurement capabilities. The component
           can measure its dimensions and report back to JavaScript.

ios/NitroTextView.swift

@@ -118,7 +118,8 @@ final class NitroTextView: UITextView {
         var index = glyphRange.location
         while index < NSMaxRange(glyphRange) {
             var lineGlyphRange = NSRange(location: 0, length: 0)
-            let usedRect = lm.lineFragmentUsedRect(forGlyphAt: index, effectiveRange: &lineGlyphRange)
+            let usedRect = lm.lineFragmentUsedRect(
+                forGlyphAt: index, effectiveRange: &lineGlyphRange)
             let charRange = lm.characterRange(forGlyphRange: lineGlyphRange, actualGlyphRange: nil)
 
             let substring = fullText.substring(with: charRange)
@@ -128,7 +129,9 @@ final class NitroTextView: UITextView {
             var desc: CGFloat = -(font?.descender ?? 0)
             var cap: CGFloat = font?.capHeight ?? 0
             var xh: CGFloat = font?.xHeight ?? 0
-            if let attrsFont = storage?.attribute(.font, at: charRange.location, effectiveRange: nil) as? UIFont {
+            if let attrsFont = storage?.attribute(
+                .font, at: charRange.location, effectiveRange: nil) as? UIFont
+            {
                 asc = attrsFont.ascender
                 desc = -(attrsFont.descender)
                 cap = attrsFont.capHeight
@@ -160,8 +163,33 @@ final class NitroTextView: UITextView {
     }
 
     @objc private func handleTap(_ recognizer: UITapGestureRecognizer) {
-        if recognizer.state == .ended {
-            nitroTextDelegate?.onNitroTextPress()
-        }
+        guard recognizer.state == .ended else { return }
+        clearSelectionIfNeeded(at: recognizer.location(in: self))
+        nitroTextDelegate?.onNitroTextPress()
     }
+
 }
+
+private extension NitroTextView {
+    func selectionLength(_ range: UITextRange) -> Int {
+        offset(from: range.start, to: range.end)
+    }
+
+    func isPoint(_ point: CGPoint, insideSelection range: UITextRange) -> Bool {
+        for selectionRect in selectionRects(for: range) {
+            let rect = selectionRect.rect
+            if rect.isNull || rect.isEmpty { continue }
+            if rect.contains(point) { return true }
+        }
+        return false
+    }
+
+    func clearSelectionIfNeeded(at point: CGPoint) {
+        guard let currentSelection = selectedTextRange,
+            selectionLength(currentSelection) > 0
+        else { return }
+        if !isPoint(point, insideSelection: currentSelection) {
+            selectedTextRange = nil
+        }
+    }
+}

src/nitro-text.tsx

@@ -29,8 +29,16 @@ type NitroTextPropsWithEvents = Pick<
 > &
   Omit<TextProps, 'onTextLayout'>
 
+let TextAncestorContext = unstable_TextAncestorContext
+if (
+  Platform.constants.reactNativeVersion.major > 0 ||
+  (Platform.constants.reactNativeVersion.major === 0 &&
+    Platform.constants.reactNativeVersion.minor >= 81)
+) {
+  TextAncestorContext = require('react-native/Libraries/Text/TextAncestor')
+}
 export const NitroText = (props: NitroTextPropsWithEvents) => {
-  const isInsideRNText = React.useContext(unstable_TextAncestorContext)
+  const isInsideRNText = React.useContext(TextAncestorContext)
   const {
     children,
     style,