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

Please move the emoji legend/key out of the Column Header #9

Closed
zersiax opened this issue Jul 25, 2021 · 2 comments · Fixed by #10
Closed

Please move the emoji legend/key out of the Column Header #9

zersiax opened this issue Jul 25, 2021 · 2 comments · Fixed by #10

Comments

@zersiax
Copy link

zersiax commented Jul 25, 2021

When the header of a column in a Markdown table gets converted to HTML , that header becomes the semantic column header for that column in the HTML table. This is usually a good thing.
One side effect of this is that screenreaders will, when encountering a cell in that column, helpfully read the column header again before the cell contents. This is done because a blind/VI computer user can only see one thing at a time, and it makes it easier to associate the cell with it's column and/or row headers. Again, this is usually a good thing.
In this table however, the column header is several lines long, with a lot of info. This means that the text Learning Links(Open Book Official Docs ClapperBoard Video PageFacingUp Article InputLatinUppercase Course) is prepended to every cell contents when the cell is encountered. This is, euphemistically said, extremely annoying :)
Love the resource, don't love the time I'm losing by this little glitch. Could you move the emoji guide out of the table, or would you accept a pull request doing this? :) Thanks for reading in any case.

@PovilasKorop
Copy link
Collaborator

@zersiax thanks for such a detailed explanation. Done and fixed in the Pull Request #10

@zersiax
Copy link
Author

zersiax commented Jul 26, 2021

Much better, thanks a lot :)

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 a pull request may close this issue.

2 participants