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 block] Showing the play button after inserting URL that doesn't exist #42443

Closed
RenanLukas opened this issue Jul 14, 2022 · 2 comments
Closed
Labels
[Block] Video Affects the Video Block Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended

Comments

@RenanLukas
Copy link

Description

When inserting a URL that doesn't exist to the video block, the play button can still be tapped (but nothing loads, since the URL doesn't exist).

Step-by-step reproduction instructions

1 - Open a post or a page
2 - Add a video block
3 - Select the option "Insert from URL"
4 - Insert a URL that doesn't exist (e.g. https://www.thisisnotavalidurl.com.br) and tap "Apply"
5 - Observe the video block

Expected behaviour

The video block shows the "ADD VIDEO" button and doesn't load anything (this happens if I insert an invalid URL, such as "aaaaaaa").

Actual behaviour

The video block shows the play button and let's us play the video, even though it never loads since the URL doesn't exist.

Screenshots or screen recording (optional)

Uploading gutenberg_video_block.mov…

WordPress information

  • WordPress version:
  • Gutenberg version:
  • Are all plugins except Gutenberg deactivated?
  • Are you using a default theme (e.g. Twenty Twenty-One)?

Device information

  • Device: iPhone SE
  • Operating system: iOS 15.4
  • WordPress app version: 20.3
@RenanLukas RenanLukas added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label Jul 14, 2022
@fluiddot fluiddot added [Type] Bug An existing feature does not function as intended [Block] Video Affects the Video Block [Priority] High Used to indicate top priority items that need quick attention labels Jul 15, 2022
@derekblank
Copy link
Member

This issue is resolved via #49858.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Video Affects the Video Block Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

3 participants