Fixed bullet drawing when changing direction

DEVSIX-6085

Author: Evgeniy Prudnikov

src/main/java/com/itextpdf/html2pdf/attach/impl/DefaultTagWorkerMapping.java

@@ -196,6 +196,7 @@ class DefaultTagWorkerMapping {
         workerMapping.putMapping(TagConstants.UL, CssConstants.INLINE, (lhs, rhs) -> new SpanTagWorker(lhs, rhs));
         workerMapping.putMapping(TagConstants.LI, CssConstants.INLINE, (lhs, rhs) -> new SpanTagWorker(lhs, rhs));
         workerMapping.putMapping(TagConstants.LI, CssConstants.INLINE_BLOCK, (lhs, rhs) -> new DivTagWorker(lhs, rhs));
+        workerMapping.putMapping(TagConstants.LI, CssConstants.BLOCK, (lhs, rhs) -> new DivTagWorker(lhs, rhs));
         workerMapping.putMapping(TagConstants.DD, CssConstants.INLINE, (lhs, rhs) -> new SpanTagWorker(lhs, rhs));
         workerMapping.putMapping(TagConstants.DT, CssConstants.INLINE, (lhs, rhs) -> new SpanTagWorker(lhs, rhs));
 

src/main/java/com/itextpdf/html2pdf/css/apply/impl/DefaultTagCssApplierMapping.java

@@ -154,6 +154,7 @@ class DefaultTagCssApplierMapping {
         mapping.putMapping(TagConstants.UL, CssConstants.INLINE, () -> new SpanTagCssApplier());
         mapping.putMapping(TagConstants.LI, CssConstants.INLINE, () -> new SpanTagCssApplier());
         mapping.putMapping(TagConstants.LI, CssConstants.INLINE_BLOCK, () -> new BlockCssApplier());
+        mapping.putMapping(TagConstants.LI, CssConstants.BLOCK, () -> new BlockCssApplier());
         mapping.putMapping(TagConstants.DD, CssConstants.INLINE, () -> new SpanTagCssApplier());
         mapping.putMapping(TagConstants.DT, CssConstants.INLINE, () -> new SpanTagCssApplier());
 

src/test/java/com/itextpdf/html2pdf/attribute/DirAttributeTest.java

@@ -74,7 +74,6 @@ public void differentDirsOfDlsTest() throws IOException, InterruptedException {
     }
 
     @Test
-    // TODO DEVSIX-5070 Process dots of ordered list items in the specified direction
     @LogMessages(messages = {
             @LogMessage(messageTemplate = IoLogMessageConstant.TYPOGRAPHY_NOT_FOUND, count = 18),
     })

src/test/java/com/itextpdf/html2pdf/element/ListItemTest.java

@@ -81,11 +81,65 @@ public void rtlListItemInsideLtrUnorderedListTest() throws IOException, Interrup
 		                                                     destinationFolder, "diff01_"));
 	}
 
+	@Test
+	@LogMessages(messages = {@LogMessage(messageTemplate = IoLogMessageConstant.TYPOGRAPHY_NOT_FOUND, count = 12)})
+	public void drawBulletRtlTest() throws IOException, InterruptedException {
+		String name = "drawBulletRtl";
+		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
+		                           new File(destinationFolder + name +".pdf"));
+		Assert.assertNull(new CompareTool().compareByContent(destinationFolder + name + ".pdf",
+		                                                     sourceFolder + "cmp_" + name + ".pdf",
+		                                                     destinationFolder, "diff01_"));
+	}
+
 	@Test
 	@LogMessages(messages = {@LogMessage(messageTemplate = IoLogMessageConstant.TYPOGRAPHY_NOT_FOUND, count = 16)})
-	public void differentListItemsInsideDifferentListsWithDifferentDirections() throws IOException, InterruptedException {
-		String name = "differentListItemsInsideDifferentListsWithDifferentDirections";
- 		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
+	public void drawBulletLtrTest() throws IOException, InterruptedException {
+		String name = "drawBulletLtr";
+		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
+		                           new File(destinationFolder + name +".pdf"));
+		Assert.assertNull(new CompareTool().compareByContent(destinationFolder + name + ".pdf",
+		                                                     sourceFolder + "cmp_" + name + ".pdf",
+		                                                     destinationFolder, "diff01_"));
+	}
+
+	@Test
+	@LogMessages(messages = {@LogMessage(messageTemplate = IoLogMessageConstant.TYPOGRAPHY_NOT_FOUND, count = 8)})
+	public void bulletsAreNotDrawnAsTheyAreInPageMarginsTest() throws IOException, InterruptedException {
+		String name = "bulletsAreNotDrawnAsTheyAreInPageMargins";
+		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
+		                           new File(destinationFolder + name +".pdf"));
+		Assert.assertNull(new CompareTool().compareByContent(destinationFolder + name + ".pdf",
+		                                                     sourceFolder + "cmp_" + name + ".pdf",
+		                                                     destinationFolder, "diff01_"));
+	}
+
+	@Test
+	@LogMessages(messages = {@LogMessage(messageTemplate = IoLogMessageConstant.TYPOGRAPHY_NOT_FOUND, count = 20)})
+	public void rltListItemWithDifferentMarginsTest() throws IOException, InterruptedException {
+		String name = "rltListItemWithDifferentMargins";
+		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
+		                           new File(destinationFolder + name +".pdf"));
+		Assert.assertNull(new CompareTool().compareByContent(destinationFolder + name + ".pdf",
+		                                                     sourceFolder + "cmp_" + name + ".pdf",
+		                                                     destinationFolder, "diff01_"));
+	}
+
+	@Test
+	@LogMessages(messages = {@LogMessage(messageTemplate = IoLogMessageConstant.TYPOGRAPHY_NOT_FOUND, count = 16)})
+	public void diffListItemsInsideDiffListsWithDiffDirectionsWithoutWidthTest() throws IOException, InterruptedException {
+		String name = "diffListItemsInsideDiffListsWithDiffDirectionsWithoutWidth";
+		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
+		                           new File(destinationFolder + name +".pdf"));
+		Assert.assertNull(new CompareTool().compareByContent(destinationFolder + name + ".pdf",
+		                                                     sourceFolder + "cmp_" + name + ".pdf",
+		                                                     destinationFolder, "diff01_"));
+	}
+
+	@Test
+	public void listItemWithBlockDisplayTest() throws IOException, InterruptedException {
+		String name = "listItemWithBlockDisplay";
+		HtmlConverter.convertToPdf(new File(sourceFolder + name + ".html"),
 		                           new File(destinationFolder + name +".pdf"));
 		Assert.assertNull(new CompareTool().compareByContent(destinationFolder + name + ".pdf",
 		                                                     sourceFolder + "cmp_" + name + ".pdf",