fix: RTL display issue in author merge interface

Author: akramcodez

openlibrary/plugins/upstream/utils.py

@@ -633,6 +633,31 @@ def add_metatag(tag: str = "meta", **attrs) -> None:
     context.metatags.append(Metatag(tag, **attrs))
 
 
+@public
+def is_rtl(text: str) -> bool:
+    """
+    Check if the given text is right-to-left (RTL) using Unicode bidirectional properties.
+
+    Args:
+        text: The text to check for RTL characters
+
+    Returns:
+        bool: True if the text contains RTL characters, False otherwise
+
+    Examples:
+        >>> is_rtl("Hello World")
+        False
+        >>> is_rtl("مدرسة العذاب")
+        True
+        >>> is_rtl("Hello مرحبا World")
+        True
+    """
+    if not text:
+        return False
+    # 'R' = Right-to-Left, 'AL' = Arabic Letter
+    return any(unicodedata.bidirectional(char) in ('R', 'AL') for char in text)
+
+
 @public
 def url_quote(text: str | bytes) -> str:
     if isinstance(text, str):

openlibrary/templates/merge/authors.html

@@ -92,10 +92,16 @@ <h1>$_("Merge Authors")</h1>
                         <p>No description.</p>
                     <ul>
                     $for doc in top.docs:
-                        <li><a href="$doc['key']" target="new" title="$_('Open in a new window')">$doc['title']</a> <span class="smaller">$ungettext('%(count)s edition', '%(count)s editions', doc['edition_count'], count=doc['edition_count']),
-                        $if doc.get('first_publish_year'):
-                            <span title="$_('First published in')">$doc['first_publish_year']</span>
-                        </span></li>
+                        <li>
+                            <a href="$doc['key']"
+                               target="new"
+                               title="$_('Open in a new window')"
+                               dir="$:cond(is_rtl(doc['title']), 'rtl', 'ltr')"
+                            >$doc['title']</a>
+                            <span class="smaller">
+                                $ungettext('%(count)s edition', '%(count)s editions', doc['edition_count'], count=doc['edition_count'])$if doc.get('first_publish_year'):, <span title="$_('First published in')">$doc['first_publish_year']</span>
+                            </span>
+                        </li>
                     </ul>
                 </div>
                 <div class="data count">