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

It's not clear how to change URL for embed. #6176

Closed
2 tasks
Shelob9 opened this issue Apr 13, 2018 · 6 comments
Closed
2 tasks

It's not clear how to change URL for embed. #6176

Shelob9 opened this issue Apr 13, 2018 · 6 comments
Assignees
Labels
[Feature] Blocks Overall functionality of blocks [Type] Enhancement A suggestion for improvement.

Comments

@Shelob9
Copy link
Contributor

Shelob9 commented Apr 13, 2018

Issue Overview

Once I have an embed block and I want to change the URL for the block, it's not super obvious how to edit that.

If I had to guess, I would say it is the link icon in the middle of the block:
screen shot 2018-04-13 at 1 29 48 pm

That's actually part of the UI for the embed caption, which is obvious I have a caption

screen shot 2018-04-13 at 1 30 04 pm

Possible Solution

  1. Hide the toolbar for the caption until the caption is not empty. This probably requires a placeholder or visible label for the caption so its clear you can enter a caption.
  2. Add a control for embed URL to advanced settings for the block.
  3. Use the pencil icon when block is focused to open a control for embed URL or focus the advanced setting for it:

screen shot 2018-04-13 at 1 51 32 pm

Todos

  • Tests
  • Documentation
@danielbachhuber danielbachhuber added [Type] Enhancement A suggestion for improvement. Needs Design Feedback Needs general design feedback. [Feature] Blocks Overall functionality of blocks labels Apr 14, 2018
@danielbachhuber danielbachhuber added this to the WordPress 5.0 milestone Apr 14, 2018
@danielbachhuber
Copy link
Member

Good feedback, @Shelob9.

I've filed to WordPress 5.0 because this doesn't appear to be a blocker for merge proposal.

@karmatosed
Copy link
Member

I think with our link UI unifications - having in toolbar - this could be solved. That said, the simpler option of a solid placeholder feels also worth exploring:

Hide the toolbar for the caption until the caption is not empty. This probably requires a placeholder or visible label for the caption so its clear you can enter a caption.

I'll flag as 'needs design' so we can iterate.

@karmatosed karmatosed added Needs Design Needs design efforts. and removed Needs Design Feedback Needs general design feedback. labels Apr 17, 2018
@mtias
Copy link
Member

mtias commented Apr 27, 2018

An edit link next to the alignment toolbar, like we have for the image block, etc, seems the consistent approach here.

@karmatosed karmatosed removed the Needs Design Needs design efforts. label Apr 27, 2018
@notnownikki notnownikki self-assigned this Jul 20, 2018
@notnownikki
Copy link
Member

Another vote for an edit link next to the alignment toolbar!

@notnownikki
Copy link
Member

Fix is up for review in #8084

@notnownikki
Copy link
Member

Fix merged!

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] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

5 participants