Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Technique F91: Related techniques are wrong #3788

Closed
umut-gultekin opened this issue Apr 22, 2024 · 1 comment · Fixed by #3821
Closed

Technique F91: Related techniques are wrong #3788

umut-gultekin opened this issue Apr 22, 2024 · 1 comment · Fixed by #3821

Comments

@umut-gultekin
Copy link

It should only be related to H43: Using id and headers attributes to associate data cells with header cells in data tables

@mbgower
Copy link
Contributor

mbgower commented May 3, 2024

I agree that a few of the techniques are only nominally related at best.

scottaohara added a commit that referenced this issue May 6, 2024
closes #3788

removes f46, g115 and g63 as they aren't directly related to not marking up table headers correctly.

The issue said all but H43 should be removed, but H51 demonstrates how to build a table with correctly marked up headers, so i see no reason why this shouldn't still be linked.  As it stands, support for the headers attribute is rather lack luster - so pointing to a generic use of setting up a table with correct usage of the th element is worth not removing.
@scottaohara scottaohara self-assigned this May 6, 2024
mbgower added a commit that referenced this issue May 28, 2024
closes #3788

removes f46, g115 and g63 as they aren't directly related to not marking
up table headers correctly.

The issue said all but H43 should be removed, but H51 demonstrates how
to build a table with correctly marked up headers, so i see no reason
why this shouldn't still be linked. As it stands, support for the
headers attribute is rather lack luster - so pointing to a generic use
of setting up a table with correct usage of the th element is worth not
removing.

Co-authored-by: Mike Gower <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants