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 link: unify link interfaces #6392

Closed
karmatosed opened this issue Apr 24, 2018 · 11 comments
Closed

Insert link: unify link interfaces #6392

karmatosed opened this issue Apr 24, 2018 · 11 comments
Labels
[Feature] UI Components Impacts or related to the UI component system [Type] Enhancement A suggestion for improvement.
Milestone

Comments

@karmatosed
Copy link
Member

Right now we have a little variation on styling for links, most have been update to the current one but lets fix those that haven't.

Here is the latest interface:

link-paragraph

This however on images right now isn't updated here:

link-image

You can see it even more, when you have both link interfaces open:

link-images-both

For buttons you get the following:

link-button

I would suggest we should unify the links and have everything use the same like we do in these examples:

link-coverimage

link-quote

@karmatosed karmatosed added the [Type] Bug An existing feature does not function as intended label Apr 24, 2018
@mtias mtias added [Type] Enhancement A suggestion for improvement. and removed [Type] Bug An existing feature does not function as intended labels May 8, 2018
@danielbachhuber danielbachhuber added this to the WordPress 5.0 milestone May 16, 2018
@danielbachhuber danielbachhuber added the Good First Issue An issue that's suitable for someone looking to contribute for the first time label May 16, 2018
@mtias
Copy link
Member

mtias commented Jul 19, 2018

@karmatosed what's the status of this one? Maybe we need specific issues for any that are wrong.

@designsimply designsimply changed the title Unify link interfaces Insert link: unify link interfaces Jul 27, 2018
@designsimply designsimply added Needs Design Needs design efforts. [Feature] UI Components Impacts or related to the UI component system labels Sep 7, 2018
@designsimply
Copy link
Member

@karmatosed would you like me to split this issue out into more specific ones or would it be better to wait for a unified design to be posted here first?

Regarding design, would it be possible to note the post type (e.g. Page, CPT name, or the date for posts) on the right for suggested links or should that be filed as a separate issue? Note: asking because it was requested in #9660.

screen shot 2017-07-29 at 17 39 06
Example screenshot from #2084 showing how post types are listed in link suggestions in the classic editor.

@karmatosed
Copy link
Member Author

@designsimply and @mtias, yes let's split this out into issues for each one. Let's first focus though on unifying over adding new features right now. We totally can later but the bigger issue is if each one doesn't do the same.

@mtias
Copy link
Member

mtias commented Oct 12, 2018

@karmatosed what is the status of this?

@karmatosed
Copy link
Member Author

We need to fix for button only:

button

@karmatosed karmatosed removed the Needs Design Needs design efforts. label Oct 15, 2018
@afercia
Copy link
Contributor

afercia commented Oct 20, 2018

@designsimply when you have a chance 🙂 has the post type / date thing been split in a separate issue? Couldn't find one. Thanks! /Cc @azaozz

@karmatosed
Copy link
Member Author

To illustrate what this needs. Right now we have this:

1

Ideally we would have the link design to be the same as the other instances:

2

@mtias
Copy link
Member

mtias commented Nov 1, 2018

Ideally we would have the link design to be the same as the other instances:

The button shows the url input on selection of the block, should it still use a dropdown menu in that case?

@karmatosed
Copy link
Member Author

karmatosed commented Nov 2, 2018

The button shows the url input on selection of the block, should it still use a dropdown menu in that case?

My feelings are yes because that's our interface for linking. It isn't perfect but right now feels a good step. For example even without text, you get the following for cover image:

2018-11-02 at 09 42

@mtias
Copy link
Member

mtias commented Nov 12, 2018

The biggest outlier seems to be the Button block. Punting this from 5.0 as improvements there can be done later.

@designsimply
Copy link
Member

I am closing this issue and deferring to #8000 as the last issue remaining for the request to unify the link interfaces for the paragraph, button, image caption, cover image, and quote blocks. See audit details below.

/hat tip @nfmohit-wpmudev and @kadencethemes for proposing solutions to #8000 in #10128 and #12738 respectively.

I audited link inserters for the following blocks:

  1. Paragraph
  2. Button
  3. Image (done in sidebar not popover)
  4. Image caption
  5. Cover image
  6. Quote

And found the following:

  1. Image block currently does not have a Urlpopover link inserter, image links are set in the sidebar and this looks to be by design (screenshot).
  2. Button block link inserter does not match the rest (screenshot).

screen shot 2019-02-01 at 11 37 27 am

I also moved the request to include different post types and the dates of posts to the suggested links to #13641. cc @afercia

@designsimply designsimply removed the Good First Issue An issue that's suitable for someone looking to contribute for the first time label Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] UI Components Impacts or related to the UI component system [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants