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

Gutenberg block #79

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

Conversation

rdm
Copy link

@rdm rdm commented Feb 21, 2020

This adds a JWPlayer gutenberg bock (an embed block) where the user can search for a video and add the jwplayer shortcode to the page.

This is important because the old system of adding the shortcode at the cursor doesn't work in the gutenberg block environment. (It doesn't work because when the jw metabox is active, the cursor gets removed in the block editor.)

rmnl and others added 10 commits January 31, 2017 15:16
Merge for release 1.6.1
Mostly working, but for some reason attributes get lost when the post is saved, despite the claim at https://developer.wordpress.org/block-editor/developers/block-api/block-attributes/ that "If no attribute source is specified, the attribute will be saved to (and read from) the block’s comment delimiter."

(Also, seeing CORS policy failure on atype-video-jwplayer for localtest.me, so something's up with the last deployment of that...)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants