-
Notifications
You must be signed in to change notification settings - Fork 320
Add change log notes for removal of 4.1.1 #4668
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for wcag2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
We do not normally record contents that change before the FPWD or between the FPWD and the publishing of the Rec that are not incorporated as part of the published rec (my emphasis in bold):
Since the change in level of Focus Visible was reverted before publication date, I think neither entry is necessary. As an example, the proposed Focus Appearance (Minimum) that was originally brought in as level AA is not recorded as moving to a AAA; we only record the addition and the name change. There is no mention of Focus Appearance (Enhanced) which also appeared in the August 2020 draft but was later removed. |
The original issue reported in #3468 turns out to have been resolved by #4123. This is why you saw neither item in the change log and needed to add both, whereas the issue description indicated that one was present without the other. As Mike has explained, removing the two opposite change log items was our preferred resolution. In the original issue, another point was raised RE the change log having no mention of 4.1.1 Parsing being marked as obsolete and removed, which first took effect in the 25 January 2023 Candidate Recommendation Draft publication. Would you be interested in repurposing this PR to add that to the change log instead? Apologies for the confusion with the subsequent PR and the various comments across this PR and the associated issue. |
@venkatvemuri28 as mentioned in the issue, the 2.4.7 changes are not needed. Are you open to making this PR an update for 4.1.1's removal? Suggested text is:
|
Updated. |
Although this is a correction, the errata documentation files are not within the TR space, so I don't believe this can be considered normative, and have tagged it as such. But I've just noticed that this PR does not change the errata files, but the change log, and that IS in the TR space. I've flagged this to be reviewed by the facilitators. |
LIVE VIEW : https://deploy-preview-4668--wcag2.netlify.app/guidelines/