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

Add Block: Embed #316

Closed
mtias opened this issue Mar 23, 2017 · 11 comments
Closed

Add Block: Embed #316

mtias opened this issue Mar 23, 2017 · 11 comments
Labels
[Feature] Blocks Overall functionality of blocks [Type] Task Issues or PRs that have been broken down into an individual action to take
Milestone

Comments

@mtias
Copy link
Member

mtias commented Mar 23, 2017

Attributes

  • Alignment: None, Left, Right, Center, Full Bleed.
  • Change url.

Markup

<!-- wp:embed -->
http://embedurl
<!-- /wp -->

States (see all)

image

image

@mtias mtias added Framework Issues related to broader framework topics, especially as it relates to javascript [Type] Task Issues or PRs that have been broken down into an individual action to take labels Mar 23, 2017
@ellatrix ellatrix self-assigned this Apr 14, 2017
@jasmussen jasmussen added this to the Prototype Parity milestone Apr 20, 2017
@aduth
Copy link
Member

aduth commented Apr 21, 2017

What's the markup of an embed block which has a caption assigned?

@mtias
Copy link
Member Author

mtias commented Apr 26, 2017

@jasmussen thoughts on caption there? I'm thinking it doesn't make sense to support caption in a generic embed block. It'd be fine on a specific video block, though.

@ellatrix
Copy link
Member

Why would it not?

@jasmussen
Copy link
Contributor

I'm thinking it doesn't make sense to support caption in a generic embed block. It'd be fine on a specific video block, though.

So long as I can make a linebreak, center and italicize some text, I don't have strong opinion.

I think in general, it makes sense for media items, but if you embed something and the resulting block is something non media related, yeah probably would feel weird with a caption.

@ellatrix ellatrix removed their assignment Apr 26, 2017
@mtias
Copy link
Member Author

mtias commented Apr 26, 2017

Yes, that's my hesitance, it'd feel spurious in some embeds and adds some visual and cognitive weight.

@ellatrix
Copy link
Member

ellatrix commented Apr 26, 2017

Somehow, I do like that all "object blocks" would have the same markup and caption support.

<figure>
  <!-- children -->
  <figcaption>Caption.</figcaption>
<figure>

Whether it's an image, gallery, embed, table, or other form of data visualisation.

@ellatrix
Copy link
Member

ellatrix commented Apr 26, 2017

In the case of oEmbed, this could still be the link to the resource. It's valid to store it in a figure, and it comes with some handy things like floating.

<figure>https://www.youtube.com/watch?v=Nl6U7UotA-M</figure>
<figure class="alignright">
  https://www.youtube.com/watch?v=Nl6U7UotA-M
  <figcaption>Caption.</figcaption>
<figure>

@jasmussen
Copy link
Contributor

I think you both make really solid arguments here.

Can we "wait and see what feels right" on this one? Or are there other ways we can get closer to a consensus? Do we have a list of all supported oembeds? Maybe we can look at the list and see if a whole bunch of them would look super weird with captions, and decide from that.

@mtias
Copy link
Member Author

mtias commented Apr 27, 2017

I don't have a strong opinion here :)

@mtias mtias removed this from the Prototype Parity milestone Apr 28, 2017
@mtias mtias added [Feature] Blocks Overall functionality of blocks and removed Framework Issues related to broader framework topics, especially as it relates to javascript labels May 3, 2017
@jasmussen jasmussen added this to the Alpha milestone May 10, 2017
@jasmussen jasmussen modified the milestones: May Week 3 (Alpha), Beta May 22, 2017
@jasmussen jasmussen removed this from the May Week 4: Beta milestone May 30, 2017
@jasmussen
Copy link
Contributor

Can we close this one as fixed now that the embed block is in?

@mtias mtias closed this as completed Jun 6, 2017
@StaggerLeee
Copy link

Maybe I am just a bit lazy, but I cannot see option to insert/embed video from local server ?
I mean upload to Media Library, and chose it same way as image/gallery.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

No branches or pull requests

5 participants