-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Comments
@karmatosed what's the status of this one? Maybe we need specific issues for any that are wrong. |
@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.
|
@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. |
@karmatosed what is the status of this? |
@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 |
The button shows the url input on selection of the block, should it still use a dropdown menu in that case? |
The biggest outlier seems to be the Button block. Punting this from 5.0 as improvements there can be done later. |
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:
And found the following:
I also moved the request to include different post types and the dates of posts to the suggested links to #13641. cc @afercia |
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:
This however on images right now isn't updated here:
You can see it even more, when you have both link interfaces open:
For buttons you get the following:
I would suggest we should unify the links and have everything use the same like we do in these examples:
The text was updated successfully, but these errors were encountered: