[GH-ISSUE #538] MD051 should support arbitrary `id` attributes, not just `<a>` tags #440

New issue

Closed

opened 2026-03-03 01:26:56 +03:00 by kerem · 1 comment

kerem commented

2026-03-03 01:26:56 +03:00

Owner

Originally created by @MarkLodato on GitHub (Jul 26, 2022).
Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/538

MD051 currently only checks <a> tags for id="..." attributes, but this requirement seems arbitrary. Any HTML element may have an id attribute to define an anchor.

It seems like this can be easily fixed by removing the check for a tags on line 58.

In my case, I have a table and use id attributes to link to specfic rows of the table:

# Example

<table>
<tr id="foo"><td>Foo<td>Foo is good
<tr id="bar"><td>Bar<td>Bar is also good
</table>

I like [foo](#foo) and [bar](#bar).

$ npx markdownlint test.md
test.md:8:8 MD051/link-fragments Link fragments should be valid [Context: "[foo](#foo) and [bar](#bar)"]
test.md:8:8 MD051/link-fragments Link fragments should be valid [Context: "[foo](#foo)"]

Would you be willing to accept a PR that fixes this, or is there some other reason why only a tags are accepted?

Originally created by @MarkLodato on GitHub (Jul 26, 2022). Original GitHub issue: https://github.com/DavidAnson/markdownlint/issues/538 MD051 currently only checks `<a>` tags for `id="..."` attributes, but this requirement seems arbitrary. Any HTML element may have an `id` attribute to define an anchor. It seems like this can be easily fixed by removing the check for `a` tags on [line 58](https://github.com/DavidAnson/markdownlint/blob/cba2ca0dbdc8f11e4b97c054380ffd52bb7aae19/lib/md051.js#L58). In my case, I have a table and use `id` attributes to link to specfic rows of the table: ```markdown # Example <table> <tr id="foo"><td>Foo<td>Foo is good <tr id="bar"><td>Bar<td>Bar is also good </table> I like [foo](#foo) and [bar](#bar). ``` ``` $ npx markdownlint test.md test.md:8:8 MD051/link-fragments Link fragments should be valid [Context: "[foo](#foo) and [bar](#bar)"] test.md:8:8 MD051/link-fragments Link fragments should be valid [Context: "[foo](#foo)"] ``` Would you be willing to accept a PR that fixes this, or is there some other reason why only `a` tags are accepted?

kerem

2026-03-03 01:26:56 +03:00

closed this issue
added the
bug

fixed in next
labels

kerem commented

2026-03-03 01:26:57 +03:00

Author

Owner

@DavidAnson commented on GitHub (Jul 26, 2022):

Yes, I would accept a PR for this! Please read the CONTRIBUTING guide first. Thank you!

@DavidAnson commented on GitHub (Jul 26, 2022): Yes, I would accept a PR for this! Please read the CONTRIBUTING guide first. Thank you!

kerem referenced this issue

2026-03-03 01:30:32 +03:00

[PR #440] [MERGED] Readme mention CLIs support pre-commit #866

kerem referenced this issue

2026-03-07 20:05:11 +03:00

[GH-ISSUE #374] Add a .pre-commit hook #2161

kerem referenced this issue

2026-03-07 20:10:13 +03:00

[PR #440] [MERGED] Readme mention CLIs support pre-commit #2714