Skip to content

Allow headers and scope on table cells in Sanitizers.TABLES - #407

Merged
jmanico merged 1 commit into
mainfrom
tables-headers-scope
Sep 8, 2026
Merged

Allow headers and scope on table cells in Sanitizers.TABLES#407
jmanico merged 1 commit into
mainfrom
tables-headers-scope

Conversation

@jmanico

@jmanico jmanico commented Sep 8, 2026

Copy link
Copy Markdown
Member

Salvages the two attributes from #326 that #378 did not already land, on a fresh branch against the current layout.

  • headers on td and th: a space-separated list of ID tokens (ASCII letters, digits, _ - . :), white-space collapsed, anything else rejected.
  • scope on th: row, col, rowgroup, colgroup, matched case-insensitively and emitted in canonical lower case. Support standard attributes in TABLES Policy #326's custom policy compared the lowercased value but returned the original.

Positive and negative tests in SanitizersTest, changelog entry, credit to @subbudvk.

Supersedes #326.

🤖 Generated with Claude Code

https://claude.ai/code/session_01BZfawfTUAFHN7cocUjcpQK

Salvages the two attributes from PR #326 that #378 did not already cover.

- headers on td and th: a space-separated list of ID tokens limited to
  ASCII letters, digits and _ - . : with white-space collapsed.
- scope on th: row, col, rowgroup or colgroup, matched case-insensitively
  and emitted in canonical lower case.  #326's policy compared the
  lowercased value but returned the original.

Positive and negative tests in SanitizersTest; changelog entry crediting
the original author.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BZfawfTUAFHN7cocUjcpQK
@jmanico
jmanico merged commit f5ce24c into main Sep 8, 2026
7 checks passed
@jmanico
jmanico deleted the tables-headers-scope branch September 8, 2026 02:24
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