Skip to content

fix: handle section metadata for live preview, quick-edit and experience workspace#971

Open
usman-khalid wants to merge 4 commits into
adobe:mainfrom
usman-khalid:sm-fix
Open

fix: handle section metadata for live preview, quick-edit and experience workspace#971
usman-khalid wants to merge 4 commits into
adobe:mainfrom
usman-khalid:sm-fix

Conversation

@usman-khalid

@usman-khalid usman-khalid commented May 27, 2026

Copy link
Copy Markdown
Contributor

Description

Section metadata is not being applied in DA's built-in live preview, Quick Edit and the Experience Workspace layout mode for sites that don't have client-side section-metadata processing, regardless of whether server side processing via helix-html-pipeline is auto-enabled (site created after May 1st, 2026) or via feature flag features.rendering.version: 2. All three generate their preview HTML via prose2aem, so we need to handle it there.

How Has This Been Tested?

Tested on https://sm-fix--da-live--usman-khalid.aem.page/ and directly on da.live with browser overrides for all three cases (quick-edit, EW, DA live preview)

Screenshots (if appropriate):

Before (EW):
image

After:
image

Also removes section-metadata from the page outline right rail in experience workspace as it's not a real block.

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@mhaack

mhaack commented May 28, 2026

Copy link
Copy Markdown
Contributor

Suggest to have this rendered directly in HTML like pipeline, see adobe/da-universal#222

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants