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: linkify overlay doesn't disappear after link is created #10244

Closed
commorancy opened this issue Sep 29, 2018 · 8 comments
Closed
Labels
[Feature] UI Components Impacts or related to the UI component system [Type] Bug An existing feature does not function as intended

Comments

@commorancy
Copy link

commorancy commented Sep 29, 2018

Describe the bug

The first two bugs apply to the Classic block type:

  1. A small linkify overlay is created on top of the screen when you click the link button. If you use the gear on the overlay to change the link settings in a bigger overlay, when you're done with creating the link on text, the small overlay doesn't disappear. It hangs around blocking the screen forever or until you linkify something else. If you use the small field and click the blue arrow, it does go away.

  2. I've had troubles with linkifying. The first time a try to linkify a word, it doesn't work after I supply a URL and name and click 'Add Link'. The overlay disappears and the text does not show a link. If I do this a second time, it does link.

The third bug applies to Paragraph block type.
3. Links in the paragraph type block don't work at all. When clicking the link icon, a new style overlay appears, but pasting in a link doesn't work at all. When moving the mouse between various block contexts, the linkify option loses track of the highlighted text and fails.

To Reproduce
Steps to reproduce the behavior:

  1. Click link icon
  2. Click the gear icon in the overlay
  3. Paste in a URL
  4. Click Open in New Window
  5. Click 'Add Link'

One of two things will happen. It either won't link at all and/or the small overlay will remain visible forever or until you attempt to link again and cancel.

Expected behavior
The link symbol should behave like the previous editor. Click it, supply a URL, click blue arrow, done. Text is linked and overlay disappears. If I go into the gear settings, the small overlay should also disappear when done.

Screenshots
stray overlay

Desktop (please complete the following information):

  • OS: MacOS X High Sierra
  • Browser: Firefox Quantum
  • Version: 61

Smartphone (please complete the following information):

  • Device: Untested
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]
@designsimply designsimply changed the title Linkify overlay doesn't disappear after link is created (2 bugs here) Insert Link: linkify overlay doesn't disappear after link is created Sep 30, 2018
@designsimply
Copy link
Member

Thank you for testing at this stage and for filing issues! I am labeling this with Needs Testing for further review and also wanted to make a quick note that, in the future, the preference is to file one issue per bug. In this case, the classic editor link toolbar problems are already filed at #7932 so we can continue discussing links in the paragraph block here.

@designsimply designsimply added the Needs Testing Needs further testing to be confirmed. label Sep 30, 2018
@commorancy
Copy link
Author

Cool. Since the bugs were all related to linkifying text, I hooked them together in this single report. In the future, I will separate them as requested, but it could lead to duplicate bugs if the underlying components that serve the feature are the same.

@designsimply
Copy link
Member

It can be a tricky balance sometimes for sure! I try to think of it as whether or not the work can be logically separated into smaller chunks that are easier for developers to work with. Never hurts to ask if you're not sure too.

@flexerd
Copy link

flexerd commented Oct 22, 2018

Ran into same issue.

@lizkarkoski
Copy link

There's another report in the WordPress.com public support forums of difficulty with using the link creator in Gutenberg. The user has included a screen shot in their post: https://en.forums.wordpress.com/topic/serious-bug-in-the-new-editor-concerning-the-link-tool/#post-3214967

@catehstn
Copy link

@designsimply there have been some updates to link handling -- can you please check all of these for repro? Thank you!

@designsimply
Copy link
Member

Adding the screenshot from https://en.forums.wordpress.com/topic/serious-bug-in-the-new-editor-concerning-the-link-tool/#post-3213221 dated November 15, 2018 11:40 pm for reference:

link-dialog

And noting that the problem where the link modal is not working properly in the classic editor is being traced at #7932. 👍

@designsimply
Copy link
Member

  1. Links in the paragraph type block don't work at all. When clicking the link icon, a new style overlay appears, but pasting in a link doesn't work at all. When moving the mouse between various block contexts, the linkify option loses track of the highlighted text and fails.

I tested adding links to a paragraph block using the following steps with WordPress 4.9.8 and Gutenberg 4.5.1 using Firefox 63.0.3 on macOS 10.13.6 and with Firefox 61 on macOS High Sierra (via browserstack.com).

  1. Copy a link from elsewhere.
  2. Create a new post and add a few paragraphs.
  3. Highlight one word in each paragraph and paste the link directly while the word is highlighted.

Result: links were added correctly while pasting a URL onto highlighted text.

screen shot 2018-11-23 at 12 21 44 pm
Seen at https://tan-otter.jurassic.ninja/wp-admin/post.php?post=89&action=edit running WordPress 4.9.8 and Gutenberg 4.5.1 using Firefox 63.0.3 on macOS 10.13.6 using Firefox 63.0.3 on macOS 10.13.6.

@commorancy several updates have been made since your original report and I think the issue you reported about the paragraph block (separate from the classic block) is working properly now. If you're still having trouble with links in paragraphs, please add a new reply and please let me know if your testing steps are any different compared to what I've noted above!

@designsimply designsimply added [Type] Bug An existing feature does not function as intended [Feature] UI Components Impacts or related to the UI component system and removed Needs Testing Needs further testing to be confirmed. labels Nov 23, 2018
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] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

5 participants