react-native-advanced-text 0.1.10 → 0.1.11

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

@@ -1,4 +1,3 @@
-// File: AdvancedTextView.kt
 package com.advancedtext
 
 import android.content.Context
@@ -9,9 +8,11 @@ import android.text.TextPaint
 import android.text.method.LinkMovementMethod
 import android.text.style.ClickableSpan
 import android.text.style.BackgroundColorSpan
+import android.text.style.ForegroundColorSpan
 import android.util.AttributeSet
 import android.util.Log
-import android.view.ContextMenu
+import android.view.ActionMode
+import android.view.Menu
 import android.view.MenuItem
 import android.view.View
 import android.widget.TextView
@@ -20,7 +21,7 @@ import com.facebook.react.bridge.ReactContext
 import com.facebook.react.uimanager.events.RCTEventEmitter
 import android.text.Selection
 
-class AdvancedTextView : TextView, View.OnCreateContextMenuListener {
+class AdvancedTextView : TextView {
 
     private val TAG = "AdvancedTextView"
 
@@ -29,31 +30,78 @@ class AdvancedTextView : TextView, View.OnCreateContextMenuListener {
     private var indicatorWordIndex: Int = -1
     private var lastSelectedText: String = ""
     private var isSelectionEnabled: Boolean = true
+    private var customActionMode: ActionMode? = null
 
     constructor(context: Context?) : super(context) { init() }
     constructor(context: Context?, attrs: AttributeSet?) : super(context, attrs) { init() }
     constructor(context: Context?, attrs: AttributeSet?, defStyleAttr: Int) : super(context, attrs, defStyleAttr) { init() }
 
-    private fun init() {
+        private fun init() {
         Log.d(TAG, "AdvancedTextView initialized")
 
-        // Set default text appearance
-        setTextColor(Color.BLACK)
+        // Set default text appearance - DON'T set black color here
         textSize = 16f
         setPadding(16, 16, 16, 16)
 
         movementMethod = LinkMovementMethod.getInstance()
         setTextIsSelectable(true)
-        setOnCreateContextMenuListener(this)
 
-        // Ensure minimum height for visibility during debugging
-        minHeight = 100
+        customSelectionActionModeCallback = object : ActionMode.Callback {
+            override fun onCreateActionMode(mode: ActionMode?, menu: Menu?): Boolean {
+                Log.d(TAG, "onCreateActionMode triggered")
+                customActionMode = mode
+                return true
+            }
+
+            override fun onPrepareActionMode(mode: ActionMode?, menu: Menu?): Boolean {
+                Log.d(TAG, "onPrepareActionMode triggered")
+                menu?.clear()
+
+                val selectionStart = selectionStart
+                val selectionEnd = selectionEnd
+
+                if (selectionStart >= 0 && selectionEnd >= 0 && selectionStart != selectionEnd) {
+                    lastSelectedText = text.subSequence(selectionStart, selectionEnd).toString()
+                    Log.d(TAG, "User selected text: '$lastSelectedText'")
+                    Log.d(TAG, "Menu options available: $menuOptions")
+
+                    menuOptions.forEachIndexed { index, option ->
+                        menu?.add(0, index, index, option)
+                    }
+
+                    sendSelectionEvent(lastSelectedText, "selection")
+                    return true
+                }
+                return false
+            }
+
+            override fun onActionItemClicked(mode: ActionMode?, item: MenuItem?): Boolean {
+                item?.let {
+                    val menuItemText = it.title.toString()
+                    Log.d(TAG, "Menu item clicked: $menuItemText")
+                    sendSelectionEvent(lastSelectedText, menuItemText)
+                    mode?.finish()
+                    return true
+                }
+                return false
+            }
+
+            override fun onDestroyActionMode(mode: ActionMode?) {
+                Log.d(TAG, "onDestroyActionMode")
+                customActionMode = null
+            }
+        }
     }
 
     fun setAdvancedText(text: String) {
         Log.d(TAG, "setAdvancedText: $text (length=${text.length})")
-        this.text = text
+
+        // Set the text first
+        super.setText(text, BufferType.SPANNABLE)
+
+        // Then apply highlights
         updateTextWithHighlights()
+
         // Force layout update
         requestLayout()
         invalidate()
@@ -89,86 +137,72 @@ class AdvancedTextView : TextView, View.OnCreateContextMenuListener {
         }
 
         val spannableString = SpannableString(textValue)
-        val words = textValue.split("\\s+".toRegex())
+
+        // Split words while preserving spaces for accurate indexing
+        val words = textValue.split("\\s+".toRegex()).filter { it.isNotEmpty() }
 
         var currentIndex = 0
         words.forEachIndexed { wordIndex, word ->
 
-            if (word.isNotEmpty()) {
-                val wordStart = textValue.indexOf(word, currentIndex)
-                if (wordStart >= 0) {
-                    val wordEnd = wordStart + word.length
-
-                    highlightedWords.find { it.index == wordIndex }?.let { highlightedWord ->
-                        val color = try {
-                            Color.parseColor(highlightedWord.highlightColor)
-                        } catch (e: IllegalArgumentException) {
-                            Log.e(TAG, "Invalid color: ${highlightedWord.highlightColor}, using yellow")
-                            Color.YELLOW
-                        }
-                        Log.d(TAG, "Applying highlight to word '$word' at index $wordIndex with color ${highlightedWord.highlightColor}")
-
-                        spannableString.setSpan(
-                            BackgroundColorSpan(color),
-                            wordStart,
-                            wordEnd,
-                            Spanned.SPAN_EXCLUSIVE_EXCLUSIVE
-                        )
+            // Find the actual position of the word in the text
+            val wordStart = textValue.indexOf(word, currentIndex)
+            if (wordStart >= 0) {
+                val wordEnd = wordStart + word.length
+
+                Log.d(TAG, "Processing word '$word' at position $wordStart-$wordEnd, index $wordIndex")
+
+                // Apply clickable span FIRST (this is important)
+                spannableString.setSpan(
+                    WordClickableSpan(wordIndex, word),
+                    wordStart,
+                    wordEnd,
+                    Spanned.SPAN_EXCLUSIVE_EXCLUSIVE
+                )
+
+                // Then apply background color for highlighted words
+                highlightedWords.find { it.index == wordIndex }?.let { highlightedWord ->
+                    val color = try {
+                        Color.parseColor(highlightedWord.highlightColor)
+                    } catch (e: IllegalArgumentException) {
+                        Log.e(TAG, "Invalid color: ${highlightedWord.highlightColor}, using yellow")
+                        Color.YELLOW
                     }
+                    Log.d(TAG, "Applying highlight to word '$word' at index $wordIndex with color ${highlightedWord.highlightColor}")
 
-                    if (wordIndex == indicatorWordIndex) {
-                        Log.d(TAG, "Applying indicator span to word '$word' at index $wordIndex")
+                    spannableString.setSpan(
+                        BackgroundColorSpan(color),
+                        wordStart,
+                        wordEnd,
+                        Spanned.SPAN_EXCLUSIVE_EXCLUSIVE
+                    )
+                }
 
-                        spannableString.setSpan(
-                            IndicatorSpan(),
-                            wordStart,
-                            wordEnd,
-                            Spanned.SPAN_EXCLUSIVE_EXCLUSIVE
-                        )
-                    }
+                // Then apply indicator span
+                if (wordIndex == indicatorWordIndex) {
+                    Log.d(TAG, "Applying indicator span to word '$word' at index $wordIndex")
 
-                    // clickable span
                     spannableString.setSpan(
-                        WordClickableSpan(wordIndex, word),
+                        IndicatorSpan(),
                         wordStart,
                         wordEnd,
                         Spanned.SPAN_EXCLUSIVE_EXCLUSIVE
                     )
-
-                    currentIndex = wordEnd
                 }
+
+                currentIndex = wordEnd
             }
         }
 
+        // Set the spannable text
         setText(spannableString, BufferType.SPANNABLE)
-        Log.d(TAG, "Text updated with spans")
-    }
-
-    override fun onCreateContextMenu(menu: ContextMenu?, v: View?, menuInfo: ContextMenu.ContextMenuInfo?) {
-        val selectionStart = selectionStart
-        val selectionEnd = selectionEnd
-
-        Log.d(TAG, "onCreateContextMenu triggered. selectionStart=$selectionStart selectionEnd=$selectionEnd")
 
-        if (selectionStart >= 0 && selectionEnd >= 0 && selectionStart != selectionEnd) {
-            lastSelectedText = text.subSequence(selectionStart, selectionEnd).toString()
-
-            Log.d(TAG, "User selected text: '$lastSelectedText'")
-            Log.d(TAG, "Menu options available: $menuOptions")
+        // Ensure movement method is still set
+        movementMethod = LinkMovementMethod.getInstance()
 
-            menu?.clear()
+        Log.d(TAG, "Text updated with spans, total spans")
+    }
 
-            menuOptions.forEachIndexed { index, option ->
-                menu?.add(0, index, index, option)?.setOnMenuItemClickListener {
-                    Log.d(TAG, "Menu item clicked: $option")
-                    onMenuItemClick(it, lastSelectedText)
-                    true
-                }
-            }
 
-            sendSelectionEvent(lastSelectedText, "selection")
-        }
-    }
 
     private fun onMenuItemClick(item: MenuItem, selectedText: String): Boolean {
         val menuItemText = menuOptions[item.itemId]
@@ -195,19 +229,25 @@ class AdvancedTextView : TextView, View.OnCreateContextMenuListener {
     }
 
     private inner class WordClickableSpan(
-        private val wordIndex: Int,
-        private val word: String
+    private val wordIndex: Int,
+    private val word: String
     ) : ClickableSpan() {
 
         override fun onClick(widget: View) {
-            Log.d(TAG, "Word clicked: '$word' (index=$wordIndex)")
-            sendWordPressEvent(word, wordIndex)
+            Log.d(TAG, "WordClickableSpan onClick triggered: '$word' (index=$wordIndex)")
+
+            // Small delay to ensure the click is processed
+            widget.post {
+                sendWordPressEvent(word, wordIndex)
+            }
         }
 
         override fun updateDrawState(ds: TextPaint) {
-            super.updateDrawState(ds)
+            // Don't call super to avoid default link styling (blue color, underline)
+            // Keep the original text appearance
             ds.color = currentTextColor
             ds.isUnderlineText = false
+            ds.bgColor = Color.TRANSPARENT
         }
     }
 
@@ -217,7 +257,7 @@ class AdvancedTextView : TextView, View.OnCreateContextMenuListener {
         }
 
         override fun updateDrawState(ds: TextPaint) {
-            ds.color = Color.RED
+
             ds.isFakeBoldText = true
             ds.isUnderlineText = false
         }

package/package.json

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