linking marked-up text generates multiple <a> tags
#360
Comments
courajs
added a commit
to courajs/mobiledoc-kit
that referenced
this issue
Jun 23, 2016
Specify behavior for adding markups that surround, are surrounded by, or are interleaved with existing markups. Failing tests for bustle#360
courajs
added a commit
to courajs/mobiledoc-kit
that referenced
this issue
Jun 23, 2016
Fixes bustle#360 We insert the new markup at a consistent index across the range. If we just push on the end of the list, it can end up in different positions of the markup stack. This results in unnecessary closing and re-opening of the markup each time it changes position. If we just push it at the beginning of the list, this causes unnecessary closing and re-opening of surrounding tags. So, we look for any tags open across the whole range, and push into the stack at the end of those.
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Example text, where "BOLD" is bolded:
one BOLD twoSelecting this text and adding a link to it will generate HTML with 3
<a>tags, e.g.:The rendered HTML should be:
This can interfere with styling and is not great for SEO.
The text was updated successfully, but these errors were encountered: