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

♿ [#2955] A11y: Remove B-tags if they are used for styling #1566

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

jiromaykin
Copy link
Contributor

@jiromaykin jiromaykin commented Jan 16, 2025

B-tags should not be used, usually these need to be replaced with a STRONG tag or a correct styling class.
Screen-reader software responds differently to B-tags and Strong tags so they are more than just a 'style' and have a difference in meaning.
Issue: https://taiga.maykinmedia.nl/project/open-inwoner/task/2955

Also: it is NOT allowed to use H2 headings if that heading has no paragraph and doesn't actually summarize any content - for accessibility software Headings have a specific meaning and are read out loud as a list with nested hierarchy.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.23%. Comparing base (ccc2b8f) to head (4e4196d).
Report is 5 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1566   +/-   ##
========================================
  Coverage    94.23%   94.23%           
========================================
  Files         1068     1068           
  Lines        39403    39403           
========================================
  Hits         37131    37131           
  Misses        2272     2272           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jiromaykin jiromaykin marked this pull request as draft January 20, 2025 09:54
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