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 G183: expand example 2 to actually illustrate the technique's point #2244

Merged
merged 2 commits into from
May 28, 2024

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented Mar 2, 2022

Closes #2243

Note that, in addition, the links to the glossary definition <a>relative luminance</a> are broken / not correctly converted https://www.w3.org/WAI/WCAG21/Techniques/general/G183 - they just end up pointing to https://www.w3.org/TR/WCAG21/#
However, I suspect this is an issue for @michael-n-cooper to look into (rather than trying to hardcode a url to the correct term definition https://www.w3.org/TR/WCAG21/#dfn-relative-luminance)

Copy link

@dd8 dd8 left a comment

Choose a reason for hiding this comment

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

That fixes issue 2243 (#2243). Thanks!

Copy link

@dd8 dd8 left a comment

Choose a reason for hiding this comment

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

The change fixes #2243

@michael-n-cooper
Copy link
Member

However, I suspect this is an issue for @michael-n-cooper to look into

#2240 will address the location of the relative luminance file

@patrickhlauke
Copy link
Member Author

Confirming that the problem with the relative link I mentioned seems to be fixed in https://www.w3.org/WAI/WCAG22/Techniques/general/G183

techniques/general/G183.html Outdated Show resolved Hide resolved
Co-authored-by: Alastair Campbell <[email protected]>
@mbgower mbgower merged commit 358f3a6 into main May 28, 2024
1 check passed
@mbgower mbgower deleted the patrickhlauke-issue2243 branch May 28, 2024 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

G183 Example 2 doesn't pass the G183 test procedure
6 participants