Skip to content
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions lighthouse-core/audits/accessibility/td-headers-attr.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ const AxeAudit = require('./axe-audit.js');
const i18n = require('../../lib/i18n/i18n.js');

const UIStrings = {
/** Title of an accesibility audit that evaluates if all table cell elements in a table that use the headers HTML attribute use it correctly to refer to cells within the same table. This title is descriptive of the successful state and is shown to users when no user action is required. */
title: 'Cells in a `<table>` element that use the `[headers]` attribute only refer ' +
'to other cells of that same table.',
/** Title of an accesibility audit that evaluates if all table cell elements in a table that use the headers HTML attribute use it correctly to refer to cells within the same table. This title is descriptive of the failing state and is shown to users when there is a failure that needs to be addressed. */
failureTitle: 'Cells in a `<table>` element that use the `[headers]` ' +
'attribute refers to other cells of that same table.',
/** Title of an accesibility audit that evaluates if all table cell elements in a table that use the headers HTML attribute use it correctly to refer to header cells within the same table. This title is descriptive of the successful state and is shown to users when no user action is required. */
title: 'Cells in a `<table>` element that use the `[headers]` attribute refer ' +
'to table cells within the same table.',
/** Title of an accesibility audit that evaluates if all table cell elements in a table that use the headers HTML attribute use it correctly to refer to header cells within the same table. This title is descriptive of the failing state and is shown to users when there is a failure that needs to be addressed. */
failureTitle: 'Cells in a `<table>` element that use the `[headers]` attribute refer ' +
'to element `id`s not found within the same table.',
/** Description of a Lighthouse audit that tells the user *why* they should try to pass. This is displayed after a user expands the section to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */
description: 'Screen readers have features to make navigating tables easier. Ensuring ' +
'`<td>` cells using the `[headers]` attribute only refer to other cells in the same ' +
Expand Down
4 changes: 2 additions & 2 deletions lighthouse-core/lib/i18n/locales/en-US.json
Original file line number Diff line number Diff line change
Expand Up @@ -276,10 +276,10 @@
"message": "Screen readers have features to make navigating tables easier. Ensuring `<td>` cells using the `[headers]` attribute only refer to other cells in the same table may improve the experience for screen reader users. [Learn more](https://web.dev/td-headers-attr/)."
},
"lighthouse-core/audits/accessibility/td-headers-attr.js | failureTitle": {
"message": "Cells in a `<table>` element that use the `[headers]` attribute refers to other cells of that same table."
"message": "Cells in a `<table>` element that use the `[headers]` attribute refer to element `id`s not found within the same table."
Copy link
Collaborator

Choose a reason for hiding this comment

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

@exterkamp is the plurliazation of a placeholder not a problem then? I noticed you did it in #9590 too

Copy link
Member Author

Choose a reason for hiding this comment

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

Shouldn't be, it's only a problem when we have a variable amount of things. If we passed in a variable like idCount and needed a variable amount of ids then it would be a problem.

Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't be, it's only a problem when we have a variable amount of things. If we passed in a variable like idCount and needed a variable amount of ids then it would be a problem.

but is there a single character (or equivalent characters) in other languages that means "make the attached word plural"?

Seems better to reword, even if it gets into plural awkwardness (and even better if we can reword to avoid the awkwardness) "refer to an element id not found within the same table" or whatever

Copy link
Member Author

Choose a reason for hiding this comment

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

Gotcha. Done!

Copy link
Collaborator

Choose a reason for hiding this comment

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

ya brendan much better worded my concern :) 👍

},
"lighthouse-core/audits/accessibility/td-headers-attr.js | title": {
"message": "Cells in a `<table>` element that use the `[headers]` attribute only refer to other cells of that same table."
"message": "Cells in a `<table>` element that use the `[headers]` attribute refer to table cells within the same table."
},
"lighthouse-core/audits/accessibility/th-has-data-cells.js | description": {
"message": "Screen readers have features to make navigating tables easier. Ensuring table headers always refer to some set of cells may improve the experience for screen reader users. [Learn more](https://web.dev/th-has-data-cells/)."
Expand Down
4 changes: 2 additions & 2 deletions lighthouse-core/lib/i18n/locales/en-XL.json
Original file line number Diff line number Diff line change
Expand Up @@ -276,10 +276,10 @@
"message": "Ŝćr̂éêń r̂éâd́êŕŝ h́âv́ê f́êát̂úr̂éŝ t́ô ḿâḱê ńâv́îǵât́îńĝ t́âb́l̂éŝ éâśîér̂. Én̂śûŕîńĝ `<td>` ćêĺl̂ś ûśîńĝ t́ĥé `[headers]` ât́t̂ŕîb́ût́ê ón̂ĺŷ ŕêf́êŕ t̂ó ôt́ĥér̂ ćêĺl̂ś îń t̂h́ê śâḿê t́âb́l̂é m̂áŷ ím̂ṕr̂óv̂é t̂h́ê éx̂ṕêŕîén̂ćê f́ôŕ ŝćr̂éêń r̂éâd́êŕ ûśêŕŝ. [Ĺêár̂ń m̂ór̂é](https://web.dev/td-headers-attr/)."
},
"lighthouse-core/audits/accessibility/td-headers-attr.js | failureTitle": {
"message": "Ĉél̂ĺŝ ín̂ á `<table>` êĺêḿêńt̂ t́ĥát̂ úŝé t̂h́ê `[headers]` át̂t́r̂íb̂út̂é r̂éf̂ér̂ś t̂ó ôt́ĥér̂ ćêĺl̂ś ôf́ t̂h́ât́ ŝám̂é t̂áb̂ĺê."
"message": "Ĉél̂ĺŝ ín̂ á `<table>` êĺêḿêńt̂ t́ĥát̂ úŝé t̂h́ê `[headers]` át̂t́r̂íb̂út̂é r̂éf̂ér̂ t́ô él̂ém̂én̂t́ `id`ŝ ńôt́ f̂óûńd̂ ẃît́ĥín̂ t́ĥé ŝám̂é t̂áb̂ĺê."
},
"lighthouse-core/audits/accessibility/td-headers-attr.js | title": {
"message": "Ĉél̂ĺŝ ín̂ á `<table>` êĺêḿêńt̂ t́ĥát̂ úŝé t̂h́ê `[headers]` át̂t́r̂íb̂út̂é ôńl̂ý r̂éf̂ér̂ t́ô ót̂h́êŕ ĉél̂ĺŝ óf̂ t́ĥát̂ śâḿê t́âb́l̂é."
"message": "Ĉél̂ĺŝ ín̂ á `<table>` êĺêḿêńt̂ t́ĥát̂ úŝé t̂h́ê `[headers]` át̂t́r̂íb̂út̂é r̂éf̂ér̂ t́ô t́âb́l̂é ĉél̂ĺŝ ẃît́ĥín̂ t́ĥé ŝám̂é t̂áb̂ĺê."
},
"lighthouse-core/audits/accessibility/th-has-data-cells.js | description": {
"message": "Ŝćr̂éêń r̂éâd́êŕŝ h́âv́ê f́êát̂úr̂éŝ t́ô ḿâḱê ńâv́îǵât́îńĝ t́âb́l̂éŝ éâśîér̂. Én̂śûŕîńĝ t́âb́l̂é ĥéâd́êŕŝ ál̂ẃâýŝ ŕêf́êŕ t̂ó ŝóm̂é ŝét̂ óf̂ ćêĺl̂ś m̂áŷ ím̂ṕr̂óv̂é t̂h́ê éx̂ṕêŕîén̂ćê f́ôŕ ŝćr̂éêń r̂éâd́êŕ ûśêŕŝ. [Ĺêár̂ń m̂ór̂é](https://web.dev/th-has-data-cells/)."
Expand Down
2 changes: 1 addition & 1 deletion lighthouse-core/test/results/sample_v2.json
Original file line number Diff line number Diff line change
Expand Up @@ -2059,7 +2059,7 @@
},
"td-headers-attr": {
"id": "td-headers-attr",
"title": "Cells in a `<table>` element that use the `[headers]` attribute only refer to other cells of that same table.",
"title": "Cells in a `<table>` element that use the `[headers]` attribute refer to table cells within the same table.",
"description": "Screen readers have features to make navigating tables easier. Ensuring `<td>` cells using the `[headers]` attribute only refer to other cells in the same table may improve the experience for screen reader users. [Learn more](https://web.dev/td-headers-attr/).",
"score": null,
"scoreDisplayMode": "notApplicable"
Expand Down
2 changes: 1 addition & 1 deletion proto/sample_v2_round_trip.json
Original file line number Diff line number Diff line change
Expand Up @@ -2529,7 +2529,7 @@
"id": "td-headers-attr",
"score": null,
"scoreDisplayMode": "notApplicable",
"title": "Cells in a `<table>` element that use the `[headers]` attribute only refer to other cells of that same table."
"title": "Cells in a `<table>` element that use the `[headers]` attribute refer to table cells within the same table."
},
"th-has-data-cells": {
"description": "Screen readers have features to make navigating tables easier. Ensuring table headers always refer to some set of cells may improve the experience for screen reader users. [Learn more](https://web.dev/th-has-data-cells/).",
Expand Down