Make WordPress Core

Opened 15 months ago

Closed 14 months ago

Last modified 9 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
15 months 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 15 months ago by aslamdoctor (previous) (diff)

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


14 months ago

#3 @Ankit K Gupta
14 months 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 14 months ago by Ankit K Gupta (previous) (diff)

#4 @Ankit K Gupta
14 months ago

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

#5 @swissspidy
9 months ago

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