fix: don't hardcode tabindex attr in table renderer (#4082)

TheNorthMemory · brc-dd · web-flow · commit aadc517c69fb · 2025-03-08T23:58:48.000+05:30
Co-authored-by: Divyansh Singh &lt;40380293+brc-dd@users.noreply.github.com&gt;
diff --git a/src/node/markdown/markdown.ts b/src/node/markdown/markdown.ts
@@ -246,8 +246,13 @@ export async function createMarkdownRenderer(
     )
     .use(lineNumberPlugin, options.lineNumbers)
 
+  const tableOpen = md.renderer.rules.table_open
   md.renderer.rules.table_open = function (tokens, idx, options, env, self) {
-    return '<table tabindex="0">\n'
+    const token = tokens[idx]
+    if (token.attrIndex('tabindex') < 0) token.attrPush(['tabindex', '0'])
+    return tableOpen
+      ? tableOpen(tokens, idx, options, env, self)
+      : self.renderToken(tokens, idx, options)
   }
 
   if (options.gfmAlerts !== false) {

Original file line number	Diff line number	Diff line change
`@@ -246,8 +246,13 @@ export async function createMarkdownRenderer(`
`246`	`246`	`)`
`247`	`247`	`.use(lineNumberPlugin, options.lineNumbers)`
`248`	`248`
	`249`	`+ const tableOpen = md.renderer.rules.table_open`
`249`	`250`	`md.renderer.rules.table_open = function (tokens, idx, options, env, self) {`
`250`		`- return '<table tabindex="0">\n'`
	`251`	`+ const token = tokens[idx]`
	`252`	`+ if (token.attrIndex('tabindex') < 0) token.attrPush(['tabindex', '0'])`
	`253`	`+ return tableOpen`
	`254`	`+ ? tableOpen(tokens, idx, options, env, self)`
	`255`	`+ : self.renderToken(tokens, idx, options)`
`251`	`256`	`}`
`252`	`257`
`253`	`258`	`if (options.gfmAlerts !== false) {`