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

link icon missing in block editor #46777

Closed
genem80 opened this issue Dec 23, 2022 · 20 comments · Fixed by #46798
Closed

link icon missing in block editor #46777

genem80 opened this issue Dec 23, 2022 · 20 comments · Fixed by #46798
Labels
[Block] Paragraph Affects the Paragraph Block [Feature] Link Editing Link components (LinkControl, URLInput) and integrations (RichText link formatting) [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release

Comments

@genem80
Copy link

genem80 commented Dec 23, 2022

Description

On 12/19/22 when I highlight text I can add a link or edit an existing link with the link icon. On 12/21/22 the link icon no longer exists when I highlight text. Version 14.8.0 was the gutenberg version. I just checked version 14.8.1 and the link icon is still missing.

Step-by-step reproduction instructions

1.Create a paragraph block
2. type some text
3. Highlight the text
4. The toolbar that pops up does not have the link icon

Screenshots, screen recording, code snippet

No response

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@t-hamano
Copy link
Contributor

I have tested with Guetnberg 14.8.1 and it appears to be working as expected.
Am I missing something?

link

@t-hamano t-hamano added the [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable label Dec 24, 2022
@smtping
Copy link

smtping commented Dec 24, 2022

I have the same problem too, here is a screenshot
image

@smtping
Copy link

smtping commented Dec 24, 2022

Well, I disabled gutenberg now and it works like before (@genem80 keep it disabled)
image

@t-hamano
Copy link
Contributor

@smtping
Thanks for the info.
However, I could not reproduce the problem in my environment even after activating the latest Gutenberg 🤔

@t-hamano t-hamano added the Needs Testing Needs further testing to be confirmed. label Dec 24, 2022
@genem80
Copy link
Author

genem80 commented Dec 24, 2022

So I deactivated gutenberg as @smtping said and the link icon returns (thanks @smtping). I'm relatively new to WP, so what am I losing by not using the gutenberg plugin. Do a base set of gutenberg block editing features come built in with WP?

@xoex
Copy link

xoex commented Dec 24, 2022

I can confirm I have this problem too, sometimes url button is exist, but most of times it's not.
I got this error in console :

register-format-type.js:86 Format "core/unknown" is already registered to handle bare tag name "a".

P.S: I can't edit other links from previous posts, It shows error formatting.

@smtping
Copy link

smtping commented Dec 24, 2022

@xoex just keep it disabled (keep the classic editor disabled too). I think that with the last update it became embedded (by default) with wordpress

@t-hamano
Copy link
Contributor

I have enabled Gutenberg version 14.8.1 on a clean install of WordPress 6.1.1 and still cannot reproduce the problem. Is there any improvement by disabling plugins other than Gutenberg and changing to the default theme?

@xoex
Copy link

xoex commented Dec 25, 2022

@xoex just keep it disabled (keep the classic editor disabled too). I think that with the last update it became embedded (by default) with wordpress

Best was rollback to previous version for me. But problem is really there,

@gyurmey2
Copy link

I confirm that the problem exists.
Gutenberg version: 14.9.20221224

@t-hamano t-hamano added the [Priority] High Used to indicate top priority items that need quick attention label Dec 26, 2022
@t-hamano
Copy link
Contributor

I have not yet found the conditions to reproduce this problem, but since it is considered a major problem, I would assign the label "[Priority] High".

@t-hamano
Copy link
Contributor

If possible, we would appreciate it if you could provide us with the following information:

  • Does this happen even if I disable all plugins except the Gutenberg plugin and change to the default theme?
  • Are there any error messages in your browser's console log? For example, like the one mentioned in this comment.

@Gierand
Copy link

Gierand commented Dec 26, 2022

@t-hamano, I noticed the same problem, but the Rank Math SEO plugin is responsible for it.

@t-hamano t-hamano removed [Priority] High Used to indicate top priority items that need quick attention Needs Testing Needs further testing to be confirmed. labels Dec 26, 2022
@t-hamano
Copy link
Contributor

t-hamano commented Dec 26, 2022

@Gierand
Thanks to you I was able to reproduce this problem!

This plugin remove the core inline link and add its own inline link control. It appears that perhaps due to #44086, we can no longer re-register format types with the same settings s that don't have className.

This problem can be reproduced in the browser console.

Without Gutenberg Plugin 14.8.1

The format type is successfully registered.

success

WordPress6.1.1 with Gutenberg Plugin 14.8.1

Format type registration fails.

fail

@genem80
Copy link
Author

genem80 commented Dec 26, 2022

I don't use the "Rank Math SEO" plugin. I do use the "All in One SEO" plugin so I don't know if the same issue occurs there. Some basic questions on plugins:

  1. Are there guidelines that all plugin developers need to follow so as not to step on each other toes
  2. Do you do extensive regression testing on new releases against a variety of theme/plugin combinations? I think it's not reasonable for users to do this type of QA for you.

@Thelmachido
Copy link

I can't reproduce the issue, however, we have 4 other reports on the Gutenberg Forums with the same issue.

  1. https://wordpress.org/support/topic/cant-add-links-to-posts-link-button-is-missing/
  2. https://wordpress.org/support/topic/clear-unknown-formatting-instead-of-a-link-button/

One of the folks noticed that the link option comes back when the Rank Math Seo plugin is deactivated so they reach out to them here: https://wordpress.org/support/topic/conflicts-with-gutenberg-cant-add-links/

@t-hamano
Copy link
Contributor

@Thelmachido
Thank you for the report.

As I understand it, the problem is that the latest Gutenberg doesn't allow registering format types that don't have a className. I expect that the users reported in the forum have probably disabled the core link functionality and are using their own link formats.

I would again mark this issue as a high priority.

@t-hamano t-hamano added the [Priority] High Used to indicate top priority items that need quick attention label Dec 27, 2022
@kathrynwp kathrynwp added [Type] Bug An existing feature does not function as intended [Feature] Link Editing Link components (LinkControl, URLInput) and integrations (RichText link formatting) [Block] Paragraph Affects the Paragraph Block [Type] Regression Related to a regression in the latest release labels Dec 27, 2022
@davilera
Copy link
Contributor

I think my PR #46798 fixes the issue. Can anybody please confirm?

@noahtallen
Copy link
Member

I verified the fix and merged it. Thanks @davilera! I'll work on creating a patch release for 14.8 now.

@noahtallen
Copy link
Member

noahtallen commented Dec 27, 2022

This should now be fixed in Gutenberg v14.8.2! Of course, please let us know if there are still issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Paragraph Affects the Paragraph Block [Feature] Link Editing Link components (LinkControl, URLInput) and integrations (RichText link formatting) [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Priority] High Used to indicate top priority items that need quick attention [Type] Bug An existing feature does not function as intended [Type] Regression Related to a regression in the latest release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants