Skip to content

Measure <th headers> - #200

Merged
tunetheweb merged 2 commits into
mainfrom
tunetheweb-patch-1
Sep 10, 2026
Merged

tunetheweb merged 2 commits into
mainfrom
tunetheweb-patch-1

Conversation

@tunetheweb

@tunetheweb tunetheweb commented Sep 10, 2026 •

Copy link
Copy Markdown
Member

We only measured <td header> usage, but this can also be used on <th> (and arguably makes more sense there!)

Test websites:

@tunetheweb tunetheweb changed the title Measure th with headers Measure <th headers> Sep 10, 2026
@github-actions

github-actions Bot commented Sep 10, 2026 •

Copy link
Copy Markdown

Super-linter summary

Language Validation result
EDITORCONFIG Pass ✅
JAVASCRIPT_ES Fail ❌
YAML Pass ✅

Super-linter detected linting errors

For more information, see the GitHub Actions workflow run

Powered by Super-linter

JAVASCRIPT_ES

/github/workspace/dist/a11y.js
  46:1  error  Parsing error: 'return' outside of function

✖ 1 problem (1 error, 0 warnings)

@github-actions

Copy link
Copy Markdown
https://almanac.httparchive.org/en/2022/

WPT result details

Changed custom metrics values:

{
  "_a11y": {
    "tables": {
      "total": 0,
      "total_with_caption": 0,
      "total_with_presentational": 0,
      "total_with_figcaption": 0
    },
    "file_extension_alts": {
      "total_elements_with_alt": 5,
      "total_elements_with_non_empty_alt": 0,
      "total_with_file_extension": 0,
      "file_extensions": {}
    },
    "title_and_alt": {
      "total_alt": 5,
      "total_title": 5,
      "total_both": 0,
      "total_alt_same_as_title": 0
    },
    "th_with_scope_attribute": {
      "total_th": 0,
      "total_with_scope": 0,
      "scopes": {}
    },
    "td_with_headers_attribute": {
      "total_tds": 0,
      "total_with_headers": 0
    },
    "th_with_headers_attribute": {
      "total_ths": 0,
      "total_with_headers": 0
    },
    "total_anchors_with_role_button": 0,
    "total_role_tab_with_selected_and_controls": 0,
    "placeholder_but_no_label": {
      "total_placeholder": 4,
      "total_no_label": 0
    },
    "divs_or_spans_as_button_or_link": {
      "total_role_button": 0,
      "total_role_link": 0,
      "total_either": 0
    },
    "screen_reader_classes": false,
    "form_control_a11y_tree": {
      "__error": {
        "stack": "TypeError: Cannot read properties of undefined (reading 'value')\n    at addControlToStats (<anonymous>:208:28)\n    at <anonymous>:252:1\n    at captureAndLogError (<anonymous>:30:8)\n    at wptCustomMetric (<anonymous>:201:25)\n    at <anonymous>:324:7",
        "message": "Cannot read properties of undefined (reading 'value')"
      }
    },
    "fieldset_radio_checkbox": {
      "total_radio": 0,
      "total_checkbox": 0,
      "total_radio_in_fieldset": 0,
      "total_checkbox_in_fieldset": 0,
      "fieldsets": []
    },
    "required_form_controls": []
  }
}
https://adrianroselli.com/demos/support-for-headers-attribute/index.html

WPT result details

Changed custom metrics values:

{
  "_a11y": {
    "tables": {
      "total": 2,
      "total_with_caption": 2,
      "total_with_presentational": 0,
      "total_with_figcaption": 0
    },
    "file_extension_alts": {
      "total_elements_with_alt": 0,
      "total_elements_with_non_empty_alt": 0,
      "total_with_file_extension": 0,
      "file_extensions": {}
    },
    "title_and_alt": {
      "total_alt": 0,
      "total_title": 0,
      "total_both": 0,
      "total_alt_same_as_title": 0
    },
    "th_with_scope_attribute": {
      "total_th": 44,
      "total_with_scope": 8,
      "scopes": {
        "col": 4,
        "row": 4
      }
    },
    "td_with_headers_attribute": {
      "total_tds": 196,
      "total_with_headers": 98
    },
    "th_with_headers_attribute": {
      "total_ths": 44,
      "total_with_headers": 12
    },
    "total_anchors_with_role_button": 0,
    "total_role_tab_with_selected_and_controls": 0,
    "placeholder_but_no_label": {
      "total_placeholder": 0,
      "total_no_label": 0
    },
    "divs_or_spans_as_button_or_link": {
      "total_role_button": 0,
      "total_role_link": 0,
      "total_either": 0
    },
    "screen_reader_classes": false,
    "form_control_a11y_tree": [],
    "fieldset_radio_checkbox": {
      "total_radio": 0,
      "total_checkbox": 0,
      "total_radio_in_fieldset": 0,
      "total_checkbox_in_fieldset": 0,
      "fieldsets": []
    },
    "required_form_controls": []
  }
}

@tunetheweb
tunetheweb merged commit f89904c into main Sep 10, 2026
6 of 8 checks passed
@tunetheweb
tunetheweb deleted the tunetheweb-patch-1 branch September 10, 2026 13:22
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