Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trim html string for newlines #1169

Closed
wants to merge 1 commit into from
Closed

Trim html string for newlines #1169

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/bygger/cypress/e2e/translations.spec.cy.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ describe('Translations', () => {
'<h3>Overskrift</h3><p>Beskrivelse av dette skjemaet med <a target="_blank" rel="noopener noreferrer" href="https://www.nav.no">lenke til NAV</a>.</p>':
'<h3></h3><p>Beskrivelse med <a target="_blank" rel="noopener noreferrer" href="https://www.nav.no/minside">lenke til minside</a>.</p>',
'<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3><p>Dette er et avsnitt</p><p>Her er et avsnitt <a target="_blank" rel="noopener noreferrer" href="https://www.vg.no">med lenke til VG</a> og her kommer en liste:</p><ul><li>Ta oppvasken</li><li>Handle <a target="_blank" rel="noopener noreferrer" href="https://www.coop.no/"><strong>matvarer</strong></a>, og <strong>vurder</strong> å <a target="_blank" rel="noopener noreferrer" href="https://www.zalando.no">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>':
'<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3><p>Dette er et avsnitt</p><p>Her er eit avsnitt <a target="_blank" rel="noopener noreferrer" href="https://www.dagogtid.no">med lenke til DAG OG TID</a>, og her er ei ei liste:</p><ul><li>Ta oppvasken</li><li><a target="_blank" rel="noopener noreferrer" href="https://www.kiwi.no/"><strong>Handle matvarer</strong></a>, og <strong>tenk på om du skal</strong> <a target="_blank" rel="noopener noreferrer" href="https://www.zalando.no">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>',
'<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3>\n<p>Dette er et avsnitt</p><p>Her er eit avsnitt <a target="_blank" rel="noopener noreferrer" href="https://www.dagogtid.no">med lenke til DAG OG TID</a>, og her er ei ei liste:</p><ul><li>Ta oppvasken</li><li><a target="_blank" rel="noopener noreferrer" href="https://www.kiwi.no/"><strong>Handle matvarer</strong></a>, og <strong>tenk på om du skal</strong> <a target="_blank" rel="noopener noreferrer" href="https://www.zalando.no">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>',
};
cy.get('[data-testid=html-translation]').should('have.length', 3);
cy.get('[data-testid=html-translation]')
Expand Down
2 changes: 1 addition & 1 deletion packages/bygger/cypress/fixtures/tekstblokk123Translations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
"i18n": {
"Du må selv skrive under på leveattesten. I tillegg må du få bekreftet attesten enten av to myndige personer (vitner) eller av en offentlig myndighet.": "Du må sjølv skrive....",
"<h3>Overskrift</h3><p>Beskrivelse av dette skjemaet med <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.nav.no\">lenke til NAV</a>.</p>": "<h2>Overskrift</h2><p>Beskrivelse av dette skjemaet med <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.nav.no\">lenke til NAV</a>.</p>",
"<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3><p>Dette er et avsnitt</p><p>Her er et avsnitt <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.vg.no\">med lenke til VG</a> og her kommer en liste:</p><ul><li>Ta oppvasken</li><li>Handle <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.coop.no/\"><strong>matvarer</strong></a>, og <strong>vurder</strong> å <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.zalando.no\">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>": "<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3><p>Dette er et avsnitt</p><p>Her er eit avsnitt <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.dagogtid.no\">med lenke til DAG OG TID</a>, og her er ei ei liste:</p><ul><li>Ta oppvasken</li><li><a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.kiwi.no/\"><strong>Handle matvarer</strong></a>, og <strong>tenk på om du skal</strong> <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.zalando.no\">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>"
"<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3><p>Dette er et avsnitt</p><p>Her er et avsnitt <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.vg.no\">med lenke til VG</a> og her kommer en liste:</p><ul><li>Ta oppvasken</li><li>Handle <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.coop.no/\"><strong>matvarer</strong></a>, og <strong>vurder</strong> å <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.zalando.no\">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>": "<h3>Tekstblokk med mye formatering og eksisterende oversettelse</h3>\n<p>Dette er et avsnitt</p><p>Her er eit avsnitt <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.dagogtid.no\">med lenke til DAG OG TID</a>, og her er ei ei liste:</p><ul><li>Ta oppvasken</li><li><a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.kiwi.no/\"><strong>Handle matvarer</strong></a>, og <strong>tenk på om du skal</strong> <a target=\"_blank\" rel=\"noopener noreferrer\" href=\"https://www.zalando.no\">kjøpe <strong>nye</strong> klær</a>.</li></ul><p>Nytt avsnitt. Ny liste (numerert denne gangen):</p><ol><li>Første prioritet</li><li>Også viktig, men ikke så viktig</li><li>Kan utsettes</li><li>Trengs egentlig ikke å gjøres</li></ol>"
}
},
"_id": "6217a16d81df996a71091d0b",
Expand Down
3 changes: 2 additions & 1 deletion packages/shared-components/src/util/html/converters/htmlString.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@ const sanitizeHtmlString = (htmlString: string, options?: SanitizeOptions): stri

const htmlString2Json = (htmlString: string): HtmlAsJsonElement => {
const sanitizedHtmlString = sanitizeHtmlString(htmlString);
const sanitizedWithNoNewlines = sanitizedHtmlString.replace(/[\n\r]/g, '');
const div = document.createElement('div');
div.innerHTML = sanitizedHtmlString;
div.innerHTML = sanitizedWithNoNewlines;
return JSON.parse(JSON.stringify(fromElement(div)));
};

Expand Down
Loading