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

Fix link and image toggle #281

Closed
wants to merge 1 commit into from
Closed

Conversation

nerg4l
Copy link

@nerg4l nerg4l commented Dec 23, 2020

Related #25, #95

I fixed the conflicts in the previous PR and played with it. I had to change it a bit because it acted weirdly when a link contained an image.

  1. It requires a negative lookbehind which was introduced in ES2018. This might changes the requirements of the lib.
  2. The replacement is not perfect because a link/image is recognise by the toolbar when the cursor is before a [ or after a ] char.

Probably a different handling would be more appropriate.

@Ionaru Ionaru closed this in c84af4c Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant