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

Insert / edit link inconsistencies across blocks #2216

Closed
afercia opened this issue Aug 4, 2017 · 2 comments
Closed

Insert / edit link inconsistencies across blocks #2216

afercia opened this issue Aug 4, 2017 · 2 comments
Labels
Needs Design Feedback Needs general design feedback.

Comments

@afercia
Copy link
Contributor

afercia commented Aug 4, 2017

The insert/edit link toolbar behaves differently on the text block, image block, and cover image block. While the blocks have different nature, the action of inserting / editing / removing a link is always the same. As a user, I just want to insert a link, change it, or remove it.

Text block insert:

  • "Apply" button: ok
  • "Unlink" button: I haven't inserted a link yet, maybe it should be disabled?

screen shot 2017-08-04 at 11 55 27

Text block edit:

  • I can edit and unlink 👍

screen shot 2017-08-04 at 11 55 40

Image block insert:

  • the UI is different: the formatting toolbar disappears and there's a "Back" button to go back to it. I'd rather simplify and make it behave the same as the text block: show the formatting toolbar and the link toolbar
  • there's an "Apply" button but no "Unlink" button. This should be consistent: always show Unlink (disabled) or never show

screen shot 2017-08-04 at 11 55 53

Image block edit:

  • this brings me directly in "edit mode", it should be consistent with the text block and show me an initial view with the "Edit" and "Unlink" buttons
  • there's no "Unlink" button: the only way to remove the link is: empty the field and click "Back" or click outside

screen shot 2017-08-04 at 11 56 08

Cover image insert:

  • clicking on the image (not on the text): the UI allows me to insert a link; seems to me this doesn't produce any actual link, even if the UI shows me there's a link inserted... is this supposed to be used to add a link on the cover image text or on the image itself? both? Unclear.

screen shot 2017-08-04 at 11 56 16

Cover image edit:

  • long links wrap in new lines (this doesn't happen on text and image blocks)

screen shot 2017-08-04 at 11 57 27

@afercia afercia added the Design label Aug 4, 2017
@karmatosed karmatosed self-assigned this Sep 24, 2017
@karmatosed karmatosed removed their assignment Oct 2, 2017
@jeffpaul
Copy link
Member

This ticket was mentioned in Slack in #core-editor by jeffpaul. View the logs.

@jeffpaul jeffpaul added the Needs Design Feedback Needs general design feedback. label Feb 22, 2018
@karmatosed
Copy link
Member

With all the work going on with links in #4909 lets close this and review after.

ceyhun pushed a commit that referenced this issue Jun 17, 2020
* Starter Page Templates - Tooltip

* Update Release Notes

* Update Gutenberg ref
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Feedback Needs general design feedback.
Projects
None yet
Development

No branches or pull requests

3 participants