Refactor

Author: setchi

Assets/Hypertext/Examples/RegexHypertext.cs

@@ -24,7 +24,7 @@ public Entry(string regexPattern, Color color, Action<string> callback)
         }
 
         /// <summary>
-        /// 正規表現にマッチした部分文字列にクリックイベントを登録します
+        /// 正規表現にマッチした部分文字列にクリックイベントリスナを登録します
         /// </summary>
         /// <param name="regexPattern">正規表現</param>
         /// <param name="onClick">クリック時のコールバック</param>
@@ -34,7 +34,7 @@ public void OnClick(string regexPattern, Action<string> onClick)
         }
 
         /// <summary>
-        /// 正規表現にマッチした部分文字列に色とクリックイベントを登録します
+        /// 正規表現にマッチした部分文字列に色とクリックイベントリスナを登録します
         /// </summary>
         /// <param name="regexPattern">正規表現</param>
         /// <param name="color">テキストカラー</param>
@@ -58,7 +58,7 @@ public override void RemoveListeners()
         /// <summary>
         /// イベントリスナを追加します
         /// テキストの変更などでイベントの再登録が必要なときにも呼び出されます
-        /// <see cref="HypertextBase.OnClick"/> を使ってクリックイベントを登録してください
+        /// <see cref="HypertextBase.OnClick"/> を使ってクリックイベントリスナを登録してください
         /// </summary>
         protected override void AddListeners()
         {

Assets/Hypertext/Scripts/HypertextBase.cs

@@ -17,10 +17,10 @@ class Span
             public readonly Action<string> Callback;
             public List<Rect> BoundingBoxes;
 
-            public Span(int startIndex, int endIndex, Color color, Action<string> callback)
+            public Span(int startIndex, int length, Color color, Action<string> callback)
             {
                 StartIndex = startIndex;
-                Length = endIndex;
+                Length = length;
                 Color = color;
                 Callback = callback;
                 BoundingBoxes = new List<Rect>();
@@ -52,7 +52,7 @@ const char
         Canvas RootCanvas => rootCanvas ?? (rootCanvas = GetComponentInParent<Canvas>());
 
         /// <summary>
-        /// 指定した部分文字列にクリックイベントを登録します
+        /// 指定した部分文字列にクリックイベントリスナを登録します
         /// </summary>
         /// <param name="startIndex">部分文字列の開始文字位置</param>
         /// <param name="length">部分文字列の長さ</param>
@@ -88,8 +88,8 @@ public virtual void RemoveListeners()
 
         /// <summary>
         /// イベントリスナを追加します
-        /// テキストの変更などでイベントの再登録が必要なときにも呼び出されます
-        /// <see cref="HypertextBase.OnClick"/> を使ってクリックイベントを登録してください
+        /// テキストの変更などでイベントリスナの再登録が必要なときにも呼び出されます
+        /// <see cref="HypertextBase.OnClick"/> を使ってクリックイベントリスナを登録してください
         /// </summary>
         protected abstract void AddListeners();
 
@@ -154,8 +154,6 @@ protected override void OnPopulateMesh(VertexHelper toFill)
                 }
             }
 
-            m_DisableFontTextureRebuiltCallback = false;
-
             var vertices = verticesPool.Get();
             toFill.GetUIVertexStream(vertices);
 
@@ -168,6 +166,8 @@ protected override void OnPopulateMesh(VertexHelper toFill)
             toFill.Clear();
             toFill.AddUIVertexTriangleStream(vertices);
             verticesPool.Release(vertices);
+
+            m_DisableFontTextureRebuiltCallback = false;
         }
 
         void GenerateHrefBoundingBoxes(ref List<UIVertex> vertices)
@@ -181,9 +181,6 @@ void GenerateHrefBoundingBoxes(ref List<UIVertex> vertices)
                 var startIndex = visibleCharIndexMap[span.StartIndex];
                 var endIndex = visibleCharIndexMap[span.StartIndex + span.Length - 1];
 
-                startIndex = Mathf.Clamp(startIndex, 0, text.Length - 1);
-                endIndex = Mathf.Clamp(endIndex, 0, text.Length - 1);
-
                 for (var textIndex = startIndex; textIndex <= endIndex; textIndex++)
                 {
                     var vertexStartIndex = textIndex * CharVerts;
@@ -312,7 +309,7 @@ void GenerateVisibleCharIndexMap(bool verticesReduced)
 
                 if (inTag)
                 {
-                    offset++;
+                    offset--;
 
                     if (character == GreaterThan)
                     {
@@ -321,15 +318,15 @@ void GenerateVisibleCharIndexMap(bool verticesReduced)
                 }
                 else if (supportRichText && character == LesserThan)
                 {
-                    offset++;
+                    offset--;
                     inTag = true;
                 }
                 else if (invisibleChars.Contains(character))
                 {
-                    offset++;
+                    offset--;
                 }
 
-                visibleCharIndexMap[i] = i - offset;
+                visibleCharIndexMap[i] = Mathf.Max(0, i + offset);
             }
         }