Skip to content

Fix video element, DEV-1536#41

Merged
iranimij merged 1 commit intomasterfrom
DEV-1536-fix-js-error
Jan 16, 2026
Merged

Fix video element, DEV-1536#41
iranimij merged 1 commit intomasterfrom
DEV-1536-fix-js-error

Conversation

@iranimij
Copy link
Contributor

@iranimij iranimij commented Jan 16, 2026

Note

Ensures the slide widget mixin safely handles slides without a video background while preserving Cookiebot-based blocking for supported platforms.

  • Adds an early return when no videoElement is found, invoking originalWidget to avoid running video/consent logic
  • Adjusts guard to only check blockVideoConsentConfig and isSupportedVideoPlatform(src) (removes !videoElement from the combined condition)
  • Affects view/frontend/web/js/slide-widget-mixin.js behavior for video backgrounds and consent blocking

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

Signed-off-by: Iman Aboheydary <iman@customgento.com>
@iranimij iranimij merged commit 50c206d into master Jan 16, 2026
37 checks passed
@sprankhub sprankhub deleted the DEV-1536-fix-js-error branch January 16, 2026 09:12
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