-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Returns invalid attributes in tr/td elements #29
Labels
💪 phase/solved
Post is done
Comments
github-actions
bot
added
👋 phase/new
Post is being triaged automatically
🤞 phase/open
Post is being triaged manually
and removed
👋 phase/new
Post is being triaged automatically
labels
Dec 19, 2021
5 tasks
Presumably a mistake I made before leaving for holiday last week. Will try to solve it tomorrow. Thanks for raising! |
This comment has been minimized.
This comment has been minimized.
Something which I assumed was internal was leaking and causing this error. Fixed! |
Trott
added a commit
to Trott/io.js
that referenced
this issue
Dec 21, 2021
This fixes a bug that causes our tooling to generate invalid HTML. (The `align` attribute on `tr` and `td` elements was ending up with the invalid value of `"none"`.) Refs: remarkjs/remark-gfm#29 Refs: syntax-tree/mdast-util-gfm-table@a28b860
nodejs-github-bot
pushed a commit
to nodejs/node
that referenced
this issue
Dec 23, 2021
This fixes a bug that causes our tooling to generate invalid HTML. (The `align` attribute on `tr` and `td` elements was ending up with the invalid value of `"none"`.) Refs: remarkjs/remark-gfm#29 Refs: syntax-tree/mdast-util-gfm-table@a28b860 PR-URL: #41260 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos
pushed a commit
to nodejs/node
that referenced
this issue
Jan 14, 2022
This fixes a bug that causes our tooling to generate invalid HTML. (The `align` attribute on `tr` and `td` elements was ending up with the invalid value of `"none"`.) Refs: remarkjs/remark-gfm#29 Refs: syntax-tree/mdast-util-gfm-table@a28b860 PR-URL: #41260 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams
pushed a commit
to nodejs/node
that referenced
this issue
Jan 31, 2022
This fixes a bug that causes our tooling to generate invalid HTML. (The `align` attribute on `tr` and `td` elements was ending up with the invalid value of `"none"`.) Refs: remarkjs/remark-gfm#29 Refs: syntax-tree/mdast-util-gfm-table@a28b860 PR-URL: #41260 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Linkgoron
pushed a commit
to Linkgoron/node
that referenced
this issue
Jan 31, 2022
This fixes a bug that causes our tooling to generate invalid HTML. (The `align` attribute on `tr` and `td` elements was ending up with the invalid value of `"none"`.) Refs: remarkjs/remark-gfm#29 Refs: syntax-tree/mdast-util-gfm-table@a28b860 PR-URL: nodejs#41260 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams
pushed a commit
to nodejs/node
that referenced
this issue
Feb 1, 2022
This fixes a bug that causes our tooling to generate invalid HTML. (The `align` attribute on `tr` and `td` elements was ending up with the invalid value of `"none"`.) Refs: remarkjs/remark-gfm#29 Refs: syntax-tree/mdast-util-gfm-table@a28b860 PR-URL: #41260 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Initial checklist
Affected packages and versions
remark-gfm@3.0.1
Link to runnable example
No response
Steps to reproduce
Run the example.js file in the README with the example.md in the README as input.
Expected behavior
Output should match the output shown in the README.
Actual behavior
The output for the table has
align="none"
attribute showing up. This appears to be an interoperability issue between rehypeStringify and remark-gfm (and really the underlying micromark-extension-gfm-table and hast-util-to-html modules).align="none"
is not valid, or at least that's my reading of https://developer.mozilla.org/en-US/docs/Web/HTML/Element/td#attr-align. Regardless,align
is deprecated and I think the right thing to do here is omit it (which is what the example output does).Here's the output I get:
Runtime
Node v17
Package manager
npm v7
OS
macOS
Build and bundle tools
No response
The text was updated successfully, but these errors were encountered: