Skip to content

add scope attr to th tag - #119

Merged
kazeburo merged 1 commit into
mainfrom
fix/add-scope-th-tag
Jul 26, 2026
Merged

add scope attr to th tag#119
kazeburo merged 1 commit into
mainfrom
fix/add-scope-th-tag

Conversation

@kazeburo

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings July 26, 2026 12:06
@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves the accessibility semantics of the status table markup by adding scope attributes to <th> elements so assistive technologies can better associate headers with data cells.

Changes:

  • Added scope="col" to day header cells in the table <thead> and <tfoot>.
  • Added scope="row" to the per-service row header cell in <tbody>.
  • Corrected mismatched closing tags in header cells (</td></th>).
Comments suppressed due to low confidence (1)

files/index.html:102

  • Same issue in the table footer header row: this sits in a column header row, so scope="row" is incorrect. Switch it to scope="col" for correct header associations.
                            <th scope="row"></th>

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread files/index.html
@kazeburo
kazeburo merged commit 8e5f49d into main Jul 26, 2026
3 checks passed
@kazeburo
kazeburo deleted the fix/add-scope-th-tag branch July 26, 2026 12:17
@github-actions github-actions Bot mentioned this pull request Jul 26, 2026
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.

2 participants