-
Notifications
You must be signed in to change notification settings - Fork 185
Embed Icons: New icons for different types of embeds #203
Comments
Just to be clear, we are exclusively referring to "oembeds" right? Assuming yes...
QuestionI recall there's inside |
Yes, oembed. 3rd parties can create their own non-oembed-blocks if they like.
I seem to recall that should work, yes, but I'm not entirely sure. It should work. Maybe worth opening a ticket? All your thoughts here are good. On this one:
Good point. But is there another way we can indicate relationship here? Like if the generic embed has that standard roundrect with media & brackets icon, can we reuse this in a small way on the subsequent specific oembeds? Like perhaps a little brackets roundrect icon in the corner? Or is there simply not room? |
My question is more: if they show only in a tab with other Oembeds and nothing else, do we need to mark them visually in a coordinated way? I'm not sure right now as I don't have full idea of the uses of these icons. If they are always shown grouped together and never mixed with other block types in lists, then I think we can safely decide to not "mark" them in coordinated way, to get more space to express the icon, thus more clarity. If they are meant to be mixed, then I'd favor strongly to mark them in some coordinated way. And I think your square idea works well, even if I'd probably suggest to try to do a "hollow" square, not filled, and see how it looks like (ligther, allows easier reuse). Bonus: as we are using SVG as sources too now, we might even mark the border with a class and color it independently... ;) |
I think it would be helpful if we met as a team about this. Can we schedule a time in #design-dashicons to make a plan? @jasmussen @folletto @field2? |
Sure. What's a good time? |
I'm open any time EST this week.
…On Mon, Jun 26, 2017 at 4:14 AM, Joen Asmussen ***@***.***> wrote:
Sure. What's a good time?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#203 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABlIlL4-szfD_s8gcikatABfmN61Agmjks5sH2h7gaJpZM4OAnIi>
.
--
Ben Dunkle
716-289-6814
[email protected]
field2.com
@empireoflight (WordPress, Twitter, Dribbble)
|
I'm interested in attending the meeting if it works for my schedule. Subscribed to this ticket to followup. Love designing icons! :) |
We could revive the weekly #design-dashicons meetings at 7pm Friday UTC. That's probably not a good time for everyone, however. What about Friday at 3pm UTC? |
Friday at 3:00pm UTC (I think that's 7:00am PST) would work for me to meet, at least a few times! :) |
Friday isn't ideal for me, but I can make 3pm utc. |
I should be able to join too. |
This fixes #203 This PR adds a generic "block" icon, necessary for Gutenberg In addition to this, it adds a generic "embed" block, as well as a few other topical embed blocks: - post embed - audio embed - photo embed - video embed
This is ready to merge into the master branch. Can someone handle this? I'm not sure how to resolve conflicts between the last PR and this one... |
Thanks, Ben, I'll do a rebase and merge tomorrow. |
Did #239 also close this one, or are we keeping this open to track a more extensive list? |
Sorry, forgot this one. Yeah we can close it, thanks 👍 |
In the Gutenberg editor, we are creating embed aliases for every oembed WordPress supports. There are a lot of them:
Don't worry, these embeds will get a separate tab in the inserter (see WordPress/gutenberg#1325) — they are mostly there so that you can search. This helps fix the "mystery meat embed discovery" issue. Did you know you can embed a WordPress post into a WordPress post? I sure didn't. I know now. Yo dawg.
However this is a lot of logos, and as we add oembeds in the future, even more will come. Keeping and maintaining logos in an icon set like this is a losing game, which is why for Gridicons we have a strict no logos rule (and put all logos into a separate set instead).
How about we create multiple embed icons. One that is an absolutely generic embed, and then multiple specific embeds, like "video embed", "post embed", things like that? Here's a quick brainstorm:
CC: @TheTracyL @field2 @folletto
The text was updated successfully, but these errors were encountered: