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

Don't use empty attribute syntax in HTML renderer #347

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

microsounds
Copy link

For context, this fixes issue #245

Fixed a regression introduced in 0.29.0.gfm.2, where cmark-gfm would emit empty data-* attributes in HTML renderer, valid as HTML5, but invalid as XHTML.

I use cmark-gfm as part of my static site preprocessor and templating engine microsounds/kagami which I use for my personal website and while it's not explicitly stated as a requirement, I strive for XHTML5 compliance, a robust subset of XHTML and HTML5 as defined by W3C and WHATWG, and empty attribute syntax (attribute minimization) is invalid XHTML, preventing me from using my web browser's built in XHTML parser for strict syntax validation.

See: github#245
Fixed a regression introduced in 0.29.0.gfm.2, where cmark-gfm would emit empty
data-* attributes in HTML renderer, valid as HTML5, but invalid as XHTML.
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.

1 participant