Link inlineToolbar not working

[foarsitter]

Thanks for providing this amazing tool. It helps us to achieve a higher level of content creation of our users.

Enabling the inlineToolbar for the paragraph tool is not working for the link tool. Any clues how to solve this?

Here is a modified version of the example.html illustrating the problem: https://gist.github.com/foarsitter/ece006ffc97d41f5d2228a16f194b1ef

[calumk]

Thanks for providing this amazing tool. It helps us to achieve a higher level of content creation of our users.

Thanks!

Enabling the inlineToolbar for the paragraph tool is not working for the link tool. Any clues how to solve this?
Here is a modified version of the example.html illustrating the problem: https://gist.github.com/foarsitter/ece006ffc97d41f5d2228a16f194b1ef

Hmm - This is strange.
I see the problem, but i cant work out whats causing it... It seems to be the same issue for the standard paragraph tool, and works when its not in a column.

I suspect there is something going on here with the way editor.js clears / deals with toolboxes. - It might be that nesting it causes some issues, or that the two instances search for the toolbar globally in the dom etc....

Ill keep looking.

[bekiroeztuerk]

Hello @calumk ,

Do you have a solution for this? I have the same problem.

[aacoelho]

I'm experiencing the same issue. Any news about this?

[sureshHARDIYA]

link_not_working.mp4

I am having same issue. Has anyone found the solution to this? Interestingly, when you copy past article with links from somewhere and paste, it sustains the link.

[calumk]

Hello all - really sorry but no i never found a solution for this

when you copy past article with links from somewhere and paste, it sustains the link.

Yes, the issue is not with the data stored, it is probably related to having multiple instances of EditorJS in the same webpage. Maybe something like popups is stored globally

[Tomnuyts]

It's indeed true that this has to do with nesting editorjs instances.
Check out this pull request for more information about this issue: codex-team/editor.js#2581

[calumk]

Thankyou for finding this issue

If you can propose a solutionor pull request to solve it somehow then I am very happy to implement

[Tomnuyts]

Problem is this can only be fixed in editorjs itself. So I forked the editorjs repo and made the necessary changes in the file to get it working. Hopefully they will approve the pull request or provide a fix themselves soon, so that I can switch back to the original repo.

[vFire]

When the link is in columns, everything is works only except the popup window will hide imediately, so make it unavailable, are there any hack to view the popup?

[vFire]

Problem is this can only be fixed in editorjs itself. So I forked the editorjs repo and made the necessary changes in the file to get it working. Hopefully they will approve the pull request or provide a fix themselves soon, so that I can switch back to the original repo.

Are there any chance that help me on how to change in the fork of editorjs? Much appreciated.

[calumk]

When the link is in columns, everything is works only except the popup window will hide imediately, so make it unavailable, are there any hack to view the popup?

At the moment, I have no solution

I /think/ the fork at : https://github.com/loucass003/editor.js/tree/next will fix this.
You could try to clone and build this fork to see - I have not done this yet.

[vFire]

I just found once click in the any column area will trigger all editorjs instances saving... Maybe this will prevent the link popup editing modal from visibility.

[Tomnuyts]

When the link is in columns, everything is works only except the popup window will hide imediately, so make it unavailable, are there any hack to view the popup?

At the moment, I have no solution

I /think/ the fork at : https://github.com/loucass003/editor.js/tree/next will fix this. You could try to clone and build this fork to see - I have not done this yet.

This fork does indeed fix the issue.

[calumk]

So then our best bet at the moment is to promote : codex-team/editor.js#2581

[calumk]

Looks like there is some progress on codex-team/editor.js#2581

Once it merges, i will test, make some updates, and push a new version

[sureshHARDIYA]

Depends on how soon PR owner makes requested changes

[DinandN]

Any news on this problem yet?

[calumk]

7 days ago there was some progress

codex-team/editor.js#2780

Looks like we might finally have a solution in the next release of editorjs

[calumk]

Hey - It looks like this finally got merged last month, so I will do some testing, check it works, and close this issue in the next few days.