From efc137db7bdc70930bb15420560682e60330cb99 Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Sun, 31 May 2020 22:24:13 +0530
Subject: [PATCH] Preserve the direction property, even though it likely wont
 work in Qt

---
 src/calibre/ebooks/metadata/book/render.py | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/calibre/ebooks/metadata/book/render.py b/src/calibre/ebooks/metadata/book/render.py
index ba9aa7d8d7..8cd3ed8c39 100644
--- a/src/calibre/ebooks/metadata/book/render.py
+++ b/src/calibre/ebooks/metadata/book/render.py
@@ -325,9 +325,11 @@ def mi_to_html(
     ans = [u'<tr id="%s" class="%s">%s</tr>'%(fieldl.replace('#', '_'),
         classname(fieldl), html) for fieldl, html in ans]
     # print '\n'.join(ans)
-    rans = u'<style>table.fields td { vertical-align:top}</style><table class="fields" '
+    direction = 'rtl' if rtl else 'ltr'
+    rans = u'<style>table.fields td { vertical-align:top}</style><table class="fields" style="direction: %s; ' % direction
     if not for_qt:
-        direction = 'rtl' if rtl else 'ltr'
+        # This causes wasted space at the edge of the table in Qt's rich text
+        # engine, see https://bugs.launchpad.net/calibre/+bug/1881488
         margin = 'left' if rtl else 'right'
-        rans += 'style="direction: {}; margin-{}: auto" '.format(direction, margin)
-    return '{}>{}</table>'.format(rans, '\n'.join(ans)), comment_fields
+        rans += 'margin-{}: auto; '.format(margin)
+    return '{}">{}</table>'.format(rans, '\n'.join(ans)), comment_fields