#514 Test confirming issues with BIDI text when justified. [ci skip]

danfickle · Jul 16, 2020 · 09d900b · 09d900b
1 parent 9891184
commit 09d900b
Show file tree

Hide file tree

Showing 2 changed files with 48 additions and 1 deletion.
diff --git a/openhtmltopdf-examples/src/main/resources/visualtest/html/text/rtl-justify.html b/openhtmltopdf-examples/src/main/resources/visualtest/html/text/rtl-justify.html
@@ -0,0 +1,38 @@
+<html dir="auto">
+<head>
+<meta charset="UTF-8"/>
+<style>
+/* UNCOMMENT BELOW TO VIEW IN BROWSER FOR COMPARISON */
+/*
+@font-face {
+  src: url(../fonts/Karla-Bold.ttf);
+  font-family: 'TestFont';
+}
+@font-face {
+  src: url(../fonts/NotoNaskhArabic-Regular.ttf);
+  font-family: 'arabic';
+}
+*/
+@page {
+  size: 1000px 600px;
+  margin: 20px;
+}
+html * {
+  font-family: 'TestFont', 'arabic', sans-serif;
+}
+body {
+  max-width: 960px;
+}
+</style>
+</head>
+<body>
+<p style="font-size: 20px;text-align: justify;">
+وترك وانتهاءً بالسيطرة أن كلّ, فقد من ثمّة قائمة بأيدي. أن قررت بتحدّي وبريطانيا تحت. ومن زهاء الثقيلة إذ. وسوء إحكام معزّزة وفي بل. هاربر الضروري أم دنو, بزمام استعملت حول في. إذ يونيو العظمى التنازلي  1234 بحق.
+
+معزّزة وبولندا ولم لم, إعمار الخاسرة إذ أخر, فسقط تغييرات واقتصار قد ومن. ان تلك وسوء الأبرياء. تصفح يرتبط ثم كلا, ومن لم أملاً يعادل وهولندا،. مسرح وصغار الإمتعاض بـ يتم, كل لمّ حادثة 1234 بالسيطرة, تحرير لبلجيكا، ومن بـ. فقد ثانية مسؤولية الأوروبية، عن, اسبوعين بريطانيا-فرنسا ما تعد, كلّ أي الثانية التقليدية.
+
+الذود الرئيسية لم حدى, وقد مرجع لفرنسا المتساقطة، أن. هُزم الأجل عن لان, فهرست الخاطفة غير لم, المنتصر بالإنزال جعل أن. مع استراليا، الإتفاقية تحت, عملية والكوري باستخدام ٣٠ يكن. أن وقد احداث  1234 إيطاليا عسكرياً, دارت الهادي هذا أم, يتم كل السادس الحيلولة. الى وعلى والقرى الخاصّة أي, دون علاقة المدن الإتحاد هو. وباءت الإكتفاء ماليزيا، بعض و.
+
+</p>
+</body>
+</html>
diff --git a/...mples/src/test/java/com/openhtmltopdf/visualregressiontests/TextVisualRegressionTest.java b/...mples/src/test/java/com/openhtmltopdf/visualregressiontests/TextVisualRegressionTest.java
@@ -494,7 +494,16 @@ public void testTableCaptionPosition() throws IOException {
     public void testArabicBiDi() throws IOException {
         assertTrue(vtester.runTest("arabic-bidi", TestSupport.WITH_ARABIC));
     }
-
+
+    /**
+     * Tests that rtl and bidi with text-align: justify works correctly.
+     */
+    @Test
+    @Ignore // RTL/LTR text runs together and the last line is incorrectly left justified.
+    public void testIssue514JustifyRtl() throws IOException {
+        assertTrue(vtester.runTest("rtl-justify", TestSupport.WITH_ARABIC));
+    }
+
     /**
      * Tests that letter-spacing property works correctly with bi-directional text.
      * Semi-related to issue 342.