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
BUG: prevent automatic inline conversion by class #9873
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! I've never noticed such a behavior. LGTM with nits.
I think this is a critical bug. So could you rebase this into the 4.3.x branch? Then I'll release a new package including this fix as 4.3.1 soon. |
@tk0miya thanks for the review! I'll update to a class-var and rebase first thing Monday :) |
b28945c
to
c947bbf
Compare
@tk0miya ready for re-review, thanks! |
Just kidding... |
c947bbf
to
836324b
Compare
Ok, copy-paste error fixed. All set. |
Merged. Thank you for your contribution! |
Feature or Bugfix
Purpose
To prevent inline nodes from being automatically coerced into specific HTML 5 elements.
This was a new feature added in docutils 0.17 and conflicts with some of the short class names added in #9023, namely "s" which is automatically coerced into a strike-through element.
Relates
See #9872 for motivation.
This is largely a drive-by effort. I don't know if this is the way you all would prefer to solve this, but I was able to get it working, and it seems like a very mild change. No hard feelings if this isn't the way you want to go.