Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Video embeds fail if embed block is converted to a reusable block #4483

Closed
niranjan-uma-shankar opened this issue Jan 15, 2018 · 10 comments
Closed
Labels
[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Type] Bug An existing feature does not function as intended

Comments

@niranjan-uma-shankar
Copy link

niranjan-uma-shankar commented Jan 15, 2018

Issue Overview

Embedding youtube videos works well. However, if you convert the block to a reusable block, the youtube player no longer gets embedded in the post. In the editor, the embed continues to show the player, though the play button doesn't work anymore.

Steps to Reproduce (for bugs)

  1. Embed any youtube video using the Embed block
  2. Convert the block to a reusable block
  3. Preview the post - the video is no longer embedded in the post
  4. Reload the editor - you can no longer play the video by clicking the play button

Expected Behavior

The video embed should render in posts, even if it's a reusable block.

Screenshots

  1. Youtube video embed rendering properly in post

screen shot 2018-01-16 at 12 05 35 am

  1. Convert video embed to a reusable block in the editor

screen shot 2018-01-16 at 12 06 01 am

  1. Reusable block successfully created. Embed shows in editor, though video can no longer be played in the embed

screen shot 2018-01-16 at 12 06 14 am

  1. Click Preview button and the video embed is no longer visible in the post page

screen shot 2018-01-16 at 12 06 25 am

@jeffpaul jeffpaul added the [Type] Bug An existing feature does not function as intended label Jan 26, 2018
@noisysocks noisysocks added the [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) label Apr 26, 2018
@ajitbohra
Copy link
Member

Works as expected on master / v3.2

@thabotswana
Copy link

I have experienced this same issue.

I added a Youtube block in a test post, and the video was displayed on the frontend. As soon as I converted the block to a Reusable Block, the video stopped displaying on the frontend, leaving just the video URL.

@davoraltman
Copy link

@ajitbohra The issue seems to be back. Can we reopen this thread?

@iamgabrielma
Copy link
Contributor

@ajitbohra Bringing up @davoraltman and @thabotswana comment, this bug seems to be back (for internal reference, affected site 1999851-zen ).

Can you re-open this issue? Otherwise we can create a new bug report, but would be duplicating info that can already be found here.

@ajitbohra
Copy link
Member

@iamgabrielma @davoraltman was able to reproduce, reopening issue.

@ajitbohra ajitbohra reopened this May 11, 2019
@tanjoymor
Copy link

Just to add to this, after adding a reusable block of almost anything, you need to convert it back to a regular block in order for it to function properly. I found this out on my own site after creating a reusable block for a "more" tag combined with protected content.

I also tested it on my test site with the youtube code and had the same behaviour. I'll let the user know about this work around for now.

@donmhico
Copy link
Contributor

The issue seems to be in core. I've created a ticket and attached a possible patch for this issue in the core. Please see - https://core.trac.wordpress.org/ticket/48034

@ajitbohra @tanjoymor @thabotswana @davoraltman @iamgabrielma @niranjan-uma-shankar - Please check if you have time. Thanks.

@susanjanec
Copy link

User reported this issue in 18596347-hc - do we have an update on this?

@noisysocks
Copy link
Member

@donmhico shared an explanation of what's happening here over in Trac. Since this will need to be fixed in Core, I'm closing this issue in favour of https://core.trac.wordpress.org/ticket/48034.

@noisysocks noisysocks added the [Type] WP Core Ticket Requires an upstream change from WordPress. Core Trac ticket should be linked. label Apr 20, 2020
@noisysocks
Copy link
Member

Change of plan here. As mentioned in #21043 (comment), changing the order of when the do_blocks will most definitely break plugins and themes. Instead let's fix this by making the Embed block a dynamic block. I'm re-opening #14608 to track this work.

@noisysocks noisysocks added [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed and removed [Type] WP Core Ticket Requires an upstream change from WordPress. Core Trac ticket should be linked. labels Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Status] Duplicate Used to indicate that a current issue matches an existing one and can be closed [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

10 participants