-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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 span wrapping all the things #1043
Conversation
Signed-off-by: Andrew Boyarshin <[email protected]>
For example? |
@lunny https://try.gitea.io/STALKER_2010/wine-staging/wiki, notice horizontal table borders. Double sized compared to the ones before my PR #186. |
And most likely, it was also the reason why |
Out of the topic. Do you know https://try.gitea.io/STALKER_2010/wine-staging/wiki the right sidebar seems not correct? |
@lunny yes, I noticed it long ago, when I was working on #186 PR version for Gogs, yet I didn't work on that. It must be tabulation (or spacing, whatever), that drives blackfriday crazy. As far as I have found out, blackfriday works with list items incorrectly when list items do not start at the beginning of the line. Most likely, it is also the cause of #411. |
trusted LGTM |
Anyone willing to review? |
LGTM |
As it is in the title. Wrap in
<span>
only if text contains link to autolink. Fixes double sized table borders. Ready as it is. Fixes regression introduced in #186.