Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#53609 closed defect (bug) (fixed)

Responsive widget block embeds not showing in the customizer

Reported by: walbo's profile walbo Owned by: noisysocks's profile noisysocks
Milestone: 5.8.1 Priority: normal
Severity: normal Version: 5.8
Component: Editor Keywords: has-patch fixed-major
Focuses: Cc:

Description

Embeds that are responsive are hidden in the customizer widgets. (ex. a YouTube embed)

Steps to reproduce:

  • Go to the widget section in the customizer.
  • Add a YouTube block and insert a video.
  • The block is inserted but the video is hidden in the sidebar.

Attachments (1)

Skjermbilde 2021-07-06 kl. 19.13.46.png (265.8 KB) - added by walbo 3 years ago.

Download all attachments as: .zip

Change History (10)

#1 @desrosj
3 years ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 5.8

Moving to 5.8 for testing.

#2 @walbo
3 years ago

  • Keywords has-patch added; needs-patch removed

Adding wp-embed-responsive to the body class seems to do the trick. Will do some testing.

#3 follow-up: @noisysocks
3 years ago

  • Keywords needs-patch added; has-patch removed

@walbo: Did you mean to upload a patch?

#4 @noisysocks
3 years ago

  • Milestone changed from 5.8 to 5.8.1

#5 in reply to: ↑ 3 @walbo
3 years ago

  • Version set to trunk

Replying to noisysocks:

@walbo: Did you mean to upload a patch?

Added a PR last night. https://github.com/WordPress/wordpress-develop/pull/1476 Not sure why it didn't add a comment/keyword in the ticket.

#6 @desrosj
3 years ago

  • Keywords has-patch added; needs-patch removed

#7 @noisysocks
3 years ago

  • Owner set to noisysocks
  • Resolution set to fixed
  • Status changed from new to closed

In 51716:

Customize: Add wp-embed-responsive class to body if using widgets block editor

Appends the wp-embed-responsive class to the body if the widgets block editor is
enabled. This fixes a bug preventing embeds (e.g. YouTube embeds) appearing in
the widgets block editor.

Fixes #53609.
Props walbo, kevin940726.

#8 @desrosj
3 years ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening to backport to 5.8 branch.

#9 @desrosj
3 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 51720:

Customize: Add wp-embed-responsive class to body if using widgets block editor

Appends the wp-embed-responsive class to the body if the widgets block editor is enabled. This fixes a bug preventing embeds (e.g. YouTube embeds) appearing in the widgets block editor.

Props walbo, kevin940726, noisysocks.
Merges [51716] to the 5.8 branch.
Fixes #53609.

Note: See TracTickets for help on using tickets.