adobe / aem-lib

Apache License 2.0
2 stars 7 forks source link

feature: rewrite markup out of pipeline to ensure all block div's contain appropriate paragraph tags #45

Closed shsteimer closed 7 months ago

shsteimer commented 8 months ago

Description

This change adds paragraph tags to block divs during decoration to avoid situations where the DOM lacks consistency depending on exactly how the block was authored.

Related Issue

Fix #44

Motivation and Context

Ideally this type of change could be made to HTML pipeline, but we can't do that for backwards compatibility reasons. By doing this here, we can slowly phase out projects that may be backwards incompatible with this type of change, and eventually (maybe?) remove it from here and add it to the pipeline.

This is likely not the only change that will eventually go into this category, which is why I did this in a new pipeline-utils file.

How Has This Been Tested?

Added test coverage. I have used similar code on projects as well

Screenshots (if appropriate):

n/a

Types of changes

Checklist:

adobe-bot commented 6 months ago

:tada: This PR is included in version 1.5.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: