Skip to content

Dm video viewers - #66

Open
priyankmodiPM wants to merge 5 commits into
aem-showcase:mainfrom
priyankmodiPM:dm-video-viewers
Open

Dm video viewers#66
priyankmodiPM wants to merge 5 commits into
aem-showcase:mainfrom
priyankmodiPM:dm-video-viewers

Conversation

@priyankmodiPM

@priyankmodiPM priyankmodiPM commented Jun 7, 2026

Copy link
Copy Markdown

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #

Test URLs:

Priyank Modi and others added 5 commits June 8, 2026 01:50
Adds a new block that parses any Scene7 viewer URL (e.g. SmartCropVideoViewer),
loads the viewer JS dynamically, and initialises the matching s7viewers class.
Registers the block in the Universal Editor component definition.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Register dm-video-viewers in UE component files and section filter,
load editor-support.js conditionally in loadLazy for preview/localhost.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…URLs

- Match dm-video iframe-wrap layout: 16/9 aspect ratio, black background,
  8px border-radius, overflow hidden, absolute-fill inner divs
- Replace toHttp() with toHttps() so all viewer/asset URLs use https

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat: align dm-video-viewers styling with dm-video and enforce https …
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.

1 participant