Skip to content

Dev 1111 fix blocked video styles#37

Merged
iranimij merged 12 commits intomasterfrom
DEV-1111-fix-blocked-video-styles
Oct 6, 2025
Merged

Dev 1111 fix blocked video styles#37
iranimij merged 12 commits intomasterfrom
DEV-1111-fix-blocked-video-styles

Conversation

@iranimij
Copy link
Contributor

@iranimij iranimij commented Oct 2, 2025

Note

Introduce a reusable video-blocker and integrate it with Fotorama and PageBuilder slides to block videos until marketing consent; fix PageBuilder iframe/video styling based on Cookiebot consent.

  • Frontend:
    • Reusable blocker: Add view/frontend/web/js/video-blocker-widget.js to render a consent blocker overlay and message for videos.
    • Fotorama integration: Refactor fotorama-video-events-mixin.js to use video-blocker-widget instead of inline DOM creation.
    • PageBuilder slides:
      • Add mixin mapping in requirejs-config.js for Magento_PageBuilder/.../slide/appearance/default/widget to CustomGento_Cookiebot/js/slide-widget-mixin.
      • New slide-widget-mixin.js blocks background videos until consent, toggles data-video-src/data-cookieblock-src, injects blocker, restores video and cleans blocker on consent, and reinitializes video-background/jarallax.
    • Iframe handling/styles: Update templates/iframe-handler.phtml script to adjust PageBuilder video container padding/background on Cookiebot events and improve blocked iframe placeholder/message rendering.

Written by Cursor Bugbot for commit 76843a9. This will update automatically on new commits. Configure here.

@iranimij iranimij requested a review from sprankhub October 2, 2025 09:50
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@iranimij iranimij requested a review from sprankhub October 2, 2025 13:21
@iranimij iranimij requested a review from sprankhub October 6, 2025 13:40
@iranimij iranimij merged commit fe3da1f into master Oct 6, 2025
7 of 37 checks passed
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