VA-352 preserve new lines in paatos

Opetushallitus · Jun 7, 2023 · 181beee · 181beee
1 parent caaf5ce
commit 181beee
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/playwright/tests/hakemuksen-arviointi/paatos.preview.test.ts b/playwright/tests/hakemuksen-arviointi/paatos.preview.test.ts
@@ -7,7 +7,10 @@ import { BrowserContext } from 'playwright-chromium'
 import { HaunTiedotPage } from '../../pages/hakujen-hallinta/HaunTiedotPage'
 
 const lisatekstiDefault = 'myönteinenlisäteksti default'
-const lisatekstiAmmatillinenKoulutus = 'myönteinenlisäteksti ammatillinen koulutus'
+const lisatekstiAmmatillinenKoulutus = `myönteinenlisäteksti
+monella rivillä
+ammatillinen koulutus
+`
 
 const test = submittedHakemusTest.extend({
   avustushakuID: async ({ page, userCache, hakuProps }, use, testInfo) => {
@@ -49,7 +52,7 @@ test('paatos lisäteksti', async ({ closedAvustushaku, context, avustushakuID, p
     await expect(taTili.value).toBeHidden()
     await openLuonnosAndExpectToContainText(page, context, lisatekstiDefault)
   })
-  await test.step('if koulutusaste selected and have text for koulutusaste show it', async () => {
+  await test.step('if koulutusaste selected and have text for koulutusaste show it (multiline text)', async () => {
     await taTili.input.click()
     await taTili.option.nth(0).click()
     await expect(taTili.value).toContainText('Ammatillinen koulutus')
@@ -74,6 +77,6 @@ const openLuonnosAndExpectToContainText = async (
     context.waitForEvent('page'),
     page.click('a:text-is("Luonnos")'),
   ])
-  await expect(paatosPage.locator('section').nth(3)).toContainText(text)
+  await expect(paatosPage.locator('section').nth(3)).toContainText(text, { useInnerText: true })
   await paatosPage.close()
 }
diff --git a/server/resources/templates/paatos.html b/server/resources/templates/paatos.html
@@ -17,7 +17,15 @@
     .section:last-of-type {border:none;}
     h1 { text-transform: uppercase; font-weight: bold; font-size: 15px; margin: 2em 0;}
     h2 {margin: 20px 0 0  ; width: 25%; float: left; font-weight: normal; font-size: 16px; text-transform: uppercase; min-width: 200px;}
-    .content { width: 75%; float: left; min-width: 400px; margin: 20px 0;}
+    .content { width: 75%;
+      float: left;
+      min-width: 400px;
+      margin: 20px 0;
+      white-space: pre-line;
+      display: flex;
+      flex-direction: column;
+      align-items: flex-start;
+    }
     p {margin-top: 0;}
     .todo { text-transform: uppercase; font-style: italic;}
     table {width: 100%; margin-bottom: 2em; border-collapse: collapse; page-break-inside: avoid;}
@@ -133,14 +141,13 @@ <h2>{{#t}}avustuslaji{{/t}}</h2>
   <section class='section'>
     <h2 data-test-id="muutoksenhaku-title">{{#t}}muutoksenhaku{{/t}}</h2>
     <div class='content'>
-      {{#t}}lisatietoja-sisalto{{/t}}
+      <p>{{#t}}lisatietoja-sisalto{{/t}}</p>
     </div>
   </section>
   <section class='section'>
     <h2 data-test-id="lisatietoja-title">{{#t}}lisatietoja{{/t}}</h2>
     <div class='content'>
-      <p>{{role.name}}, {{role.email}}<br/>
-         {{#t}}puhelin{{/t}} 029 533 1000 ({{#t}}keskus{{/t}}).
+      <p>{{role.name}}, {{role.email}}<br/>{{#t}}puhelin{{/t}} 029 533 1000 ({{#t}}keskus{{/t}}).
       </p>
       <div class="signature">{{johtaja}}</div>
       <div class="signature">{{valmistelija}}</div>