Fixing atom to capitalize text (#13792)

* Fixing atom to capitalize text Credit to @whimboo Fixes #13779 * Adding a test
SeleniumHQ · Apr 10, 2024 · 33c6b78 · 33c6b78
1 parent d142ce0
commit 33c6b78
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/javascript/atoms/dom.js b/javascript/atoms/dom.js
@@ -1173,7 +1173,7 @@ bot.dom.appendVisibleTextLinesFromTextNode_ = function(textNode, lines,
 
   if (textTransform == 'capitalize') {
     // the unicode regex ending with /gu does not work in IE
-    var re = goog.userAgent.IE ? /(^|\s|\b)(\S)/g : /(^|[^\d\p{L}\p{S}])([\p{Ll}|\p{S}])/gu;
+    var re = goog.userAgent.IE ? /(^|\s|\b)(\S)/g : /(^|\s|\b)(\S)/gu;
     text = text.replace(re, function() {
       return arguments[1] + arguments[2].toUpperCase();
     });

diff --git a/javascript/atoms/test/text_test.html b/javascript/atoms/test/text_test.html
@@ -280,6 +280,8 @@
       assertEquals('(This) “Is” [A] –Short– -Test- «For» *The* _Css_ ¿Capitalize? ?¡Transfor', text);
       text = getVisibleTextByElementId("capitalized-8");
       assertEquals('The Dutch Word: "Ijsland" Starts With A Digraph', text);
+      text = getVisibleTextByElementId("capitalized-9");
+      assertEquals('Test_text', text);
     }
 
     function getVisibleTextByElementId(id) {
@@ -430,6 +432,7 @@
     <a id="capitalized-6" style="text-transform: capitalize">ⓐⓑⓒ (ⓓⓔⓕ) —ⓖⓗⓘ— ⓙkl</a><br/>
     <a id="capitalized-7" style="text-transform: capitalize">(this) “is” [a] –short– -test- «for» *the* _css_ ¿capitalize? ?¡transfor</a><br/>
     <a id="capitalized-8" style="text-transform: capitalize">The Dutch word: "ijsland" starts with a digraph</a><br/>
+    <a id="capitalized-9" style="text-transform: capitalize">test_text</a><br/>
   </div>
 
 </body>