Merge pull request #13665 from calixteman/reserve

XFA - Fix indentation for justified paragraph

src/core/xfa/html_utils.js

@@ -529,12 +529,10 @@ function fixTextIndent(styles) {
   }
 
   // If indent is negative then it's a hanging indent.
-  const align = styles.textAlign || "left";
-  if (align === "left" || align === "right") {
-    const name = "padding" + (align === "left" ? "Left" : "Right");
-    const padding = getMeasurement(styles[name], "0px");
-    styles[name] = `${padding - indent}px`;
-  }
+  const align = styles.textAlign === "right" ? "right" : "left";
+  const name = "padding" + (align === "left" ? "Left" : "Right");
+  const padding = getMeasurement(styles[name], "0px");
+  styles[name] = `${padding - indent}px`;
 }
 
 function setAccess(node, classNames) {

src/core/xfa/template.js

@@ -897,7 +897,7 @@ class Caption extends XFAObject {
       "inactive",
       "invisible",
     ]);
-    this.reserve = getMeasurement(attributes.reserve);
+    this.reserve = Math.ceil(getMeasurement(attributes.reserve));
     this.use = attributes.use || "";
     this.usehref = attributes.usehref || "";
     this.extras = null;

test/pdfs/xfa_issue13633.pdf.link

@@ -0,0 +1 @@
+https://web.archive.org/web/20210703155102/https://www.jobcenter-remscheid.de/files/jobcenter-remscheid/download/anzeige-kug101_ba013134.pdf

test/test_manifest.json

@@ -1106,6 +1106,14 @@
        "enableXfa": true,
        "type": "eq"
     },
+    {  "id": "xfa_issue13633",
+       "file": "pdfs/xfa_issue13633.pdf",
+       "md5": "e5b0d09285ca6a140eba08d740be0ea0",
+       "link": true,
+       "rounds": 1,
+       "enableXfa": true,
+       "type": "eq"
+    },
     {  "id": "xfa_issue13631",
        "file": "pdfs/xfa_issue13631.pdf",
        "md5": "bf24397950dc8b8fcb7e91299af1315a",