Skip to content

Fix normal and large text sizing definitions in contrast-master to match WCAG SC 1.4.3 - #206

Closed
AdamMagued wants to merge 1 commit into
Community-Access:mainfrom
AdamMagued:fix/contrast-master-wcag-text-sizing
Closed

AdamMagued wants to merge 1 commit into
Community-Access:mainfrom
AdamMagued:fix/contrast-master-wcag-text-sizing

Conversation

@AdamMagued

Copy link
Copy Markdown

Resolves #198

Summary of Change

In skills/contrast-master/SKILL.md, the text sizing thresholds previously defined normal text as (under 18px or under 14px bold) and large text as (18px+ or 14px+ bold).

As reported in #198, this conflated point (pt) values with CSS pixel (px) values. Under WCAG SC 1.4.3:

  • Large scale text is defined as at least 18pt or 14pt bold.
  • Since 1pt is equivalent to approximately 1.333 CSS pixels (as noted in the WCAG SC definition), 18pt corresponds to approximately 24px, and 14pt corresponds to approximately 18.5px (or 18.67px).

This updates the thresholds in contrast-master/SKILL.md to accurately reference both pixel and point equivalents, aligning with existing reference documentation in the repository.

Verification

  • Ran npm run verify:skills (validated 108 skills)
  • Ran npm run verify:conformance (frontmatter conformant)
  • Ran npm run verify:mcp (107/107 tests passed)
  • Ran npm run verify:findings, npm run verify:report, and npm run verify:budgets (all passed)

@AdamMagued

Copy link
Copy Markdown
Author

closing this one in favor of #214 where i updated the full reference docs and validation checklist alongside the skill. sorry for the duplicate PR!

@AdamMagued AdamMagued closed this Sep 30, 2026
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.

Normal text and large text sizing values are incorrect based on WCAG SC 1.4.3 definition

1 participant