@medicus.ai/medicus-report-pdf-generator 1.3.17 → 1.3.18

package/lib/wellbeing_report_generator.js

@@ -929,9 +929,13 @@ const renderPatientTable = (profileInfo, startIndex = 4) => {
 /**
  * Render doctor quiz details
  * @param {array} doctorData - Doctor data array
+ * @param {boolean} startQuizAnswersOnNewPage - when true, force the "Quiz Answers"
+ *        title onto a fresh page instead of letting it orphan at the bottom of the
+ *        previous one. Only set for the Maison Sante quiz-only doctor PDF (no smart
+ *        report); the report-backed doctor PDF passes false so its layout is untouched.
  * @returns {string} Rendered doctor details HTML
  */
-const renderDoctorDetails = (doctorData) => {
+const renderDoctorDetails = (doctorData, startQuizAnswersOnNewPage = false) => {
     if (empty(doctorData)) {
         return '';
     }
@@ -962,27 +966,42 @@ const renderDoctorDetails = (doctorData) => {
         `;
     }
 
-    return `
-        <p class="resume-title">Quiz Answers</p>
-        <!-- display:flow-root overrides the class's flex-direction:column. Chrome 73
-             (puppeteer 1.15) does not fragment inside flex containers, so
-             break-inside:avoid on a row is ignored while the rows are flex items
-             and a row can be sliced across a page boundary. As block boxes they
-             honour it. flow-root rather than block: block lets the first row's
-             margin-top:1% collapse out through the column, shifting the whole section
-             up ~9px, which changes where the natural page break lands and spills rows
-             onto the previous page. flow-root establishes a block formatting context so
-             the margin is contained and the geometry matches the original exactly.
-             width:100% on the rows is required too: a row is a
-             block-level flex container and Chrome computes its width as 0 here
-             once it is no longer stretched by the flex parent, which collapses the
-             whole section. Measured: baseline row 907px, block-only 0px,
-             block+width:100% 907px. Inline so only these two columns change --
-             .details-column and .details-row are shared with renderPatientTable
-             and the patient-table.html blocks of maisonsante, Mediclinic and Pha. -->
+    // display:flow-root overrides the class's flex-direction:column. Chrome 73
+    // (puppeteer 1.15) does not fragment inside flex containers, so
+    // break-inside:avoid on a row is ignored while the rows are flex items
+    // and a row can be sliced across a page boundary. As block boxes they
+    // honour it. flow-root rather than block: block lets the first row's
+    // margin-top:1% collapse out through the column, shifting the whole section
+    // up ~9px, which changes where the natural page break lands and spills rows
+    // onto the previous page. flow-root establishes a block formatting context so
+    // the margin is contained and the geometry matches the original exactly.
+    // width:100% on the rows is required too: a row is a
+    // block-level flex container and Chrome computes its width as 0 here
+    // once it is no longer stretched by the flex parent, which collapses the
+    // whole section. Measured: baseline row 907px, block-only 0px,
+    // block+width:100% 907px. Inline so only these two columns change --
+    // .details-column and .details-row are shared with renderPatientTable
+    // and the patient-table.html blocks of maisonsante, Mediclinic and Pha.
+    const columnsHTML = `
         <div class="details-column" style="display:flow-root">${details1HTML}</div>
         <div class="details-column" style="display:flow-root;margin-left: 1%">${details2HTML}</div>
     `;
+
+    // Quiz-only: the caller turns .doctor-details into a page-break-inside:avoid block so
+    // the whole Quiz Answers section is pushed onto a fresh page (see the caller for why
+    // that mechanism is used instead of page-break-before). Once .doctor-details is a
+    // block rather than the shared flex container, the two columns need their own flex
+    // row to stay side by side, mirroring the gap/flex-wrap the .doctor-details flex rule
+    // normally supplies. The report-backed doctor PDF keeps the shared flex layout
+    // untouched.
+    const bodyHTML = startQuizAnswersOnNewPage
+        ? `<div style="display:flex; flex-wrap:wrap; gap:25px; width:100%;">${columnsHTML}</div>`
+        : columnsHTML;
+
+    return `
+        <p class="resume-title">Quiz Answers</p>
+        ${bodyHTML}
+    `;
 };
 
 /**
@@ -1355,8 +1374,32 @@ let generateHTMLWellbeingReportWithSmartReport = async (data, isDebugging, clien
 
     // Render doctor details
     if (!empty(doctorData)) {
-        const doctorDetailsHtml = renderDoctorDetails(doctorData);
+        // Quiz-only Maison Sante doctor PDF has no smart report above the Quiz Answers
+        // block, so its title would orphan at the bottom of page 1; start it on a fresh
+        // page there. The report-backed doctor PDF keeps its existing pagination.
+        const startQuizAnswersOnNewPage = empty(smartReport)
+            && client.toLowerCase() === 'maisonsante'
+            && !!data.IsDoctor;
+        const doctorDetailsHtml = renderDoctorDetails(doctorData, startQuizAnswersOnNewPage);
         $('.doctor-details').html(doctorDetailsHtml);
+        // Quiz-only: start the whole Quiz Answers block on a fresh page so its title
+        // does not orphan at the bottom of page 1. Chrome 73 (puppeteer 1.15) ignores
+        // page-break-before here (both on the flex container and its items), so this
+        // reuses the mechanism the .new-page sections rely on: a block that establishes
+        // its own formatting context (display:block + overflow:hidden) and refuses to be
+        // split (page-break-inside:avoid) is pushed to the next page when it cannot fit
+        // in the space left on the current one. display:block also restores the columns'
+        // side-by-side layout (they carry their own flex row now). Inline so only this
+        // render changes; the shared .doctor-details flex rule is untouched for the
+        // report-backed doctor PDF and every other client.
+        if (startQuizAnswersOnNewPage) {
+            $('.doctor-details').css({
+                'display': 'block',
+                'overflow': 'hidden',
+                'page-break-inside': 'avoid',
+                'break-inside': 'avoid'
+            });
+        }
     }
 
     // Render Big Integral Questionnaire section (Maison Sante doctor PDF only)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@medicus.ai/medicus-report-pdf-generator",
-  "version": "1.3.17",
+  "version": "1.3.18",
   "description": "Nasco corporate report - latest update in 12/10/2023 - Fix HRC for bionext",
   "main": "index.js",
   "scripts": {