Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 22 months ago

#59205 closed defect (bug) (wontfix)

Loom embed support

Reported by: kartik-suthar's profile Kartik Suthar Owned by:
Milestone: Priority: normal
Severity: normal Version: 6.3
Component: Embeds Keywords:
Focuses: ui-copy Cc:

Description

When pasting the loom video URL it will convert to an Embed block automatically
https://prnt.sc/5UbtHhODMnJy

But it will not be rendered properly on the front-end and creates a white space:
https://prnt.sc/n3BxCpdYEacQ

Change History (5)

#1 @aslamdoctor
2 years ago

This is because of the below attribute added to the embed iframes added by WP.

sandbox="allow-scripts"

If we add custom html block without this attribute, the loom embed works.

e.g.

<iframe class="wp-embedded-content" security="restricted" title="Loom Use Case: Saying Congratulations " src="https://www.loom.com/embed/cd7cf15849574e608a696a1c6646f043#?secret=xfpTfjVrI0" data-secret="xfpTfjVrI0" frameborder="0" width="640" height="480"></iframe>
Last edited 2 years ago by aslamdoctor (previous) (diff)

This ticket was mentioned in Slack in #core-test by ankit-k-gupta. View the logs.


2 years ago

#3 @Ankit K Gupta
2 years ago

Hello @kartik-suthar Welcome to trac and thanks for creating this ticket.

WordPress has added a "sandbox" attribute to iframes for enhanced security. Loom attempts to read the cookie, which is not permitted when the sandbox mode is enabled.

To resolve this issue, you should reach out to Loom.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe

Based on our [discussion] https://wordpress.slack.com/archives/C03B0H5J0/p1693935963459029, we have decided to close this matter in alignment with WordPress standards.

Last edited 2 years ago by Ankit K Gupta (previous) (diff)

#4 @Ankit K Gupta
2 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

#5 @swissspidy
22 months ago

  • Milestone Awaiting Review deleted
Note: See TracTickets for help on using tickets.