-
Notifications
You must be signed in to change notification settings - Fork 276
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
Update user-inactivity.html #4122
base: main
Are you sure you want to change the base?
Conversation
Added a css "note" class to the paragraph following the definition.
✅ Deploy Preview for wcag2 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
put in correct alphabetical order
Discussed and approved on backlog call 11/1. |
Cross linking #4149, I raised an issue after reviewing all the definitions. |
Discussed on backlog call 1/7. Mike agreed to split into two PRs since correcting alphabetical order is an obvious fix. The class change is also editorial, but use of NOTES in spec is a sensitive topic. Stays in For discussion. |
Removing change to index.html (now covered in #4256 ) to make it easier to differentiate between the scale of changes for the two changes to the definitions proposed.
241494c
to
1407529
Compare
Added a css "note" class to the paragraph following the definition.
Resolves #4117
Note: the reorder for alphabetically listing the definitions, that was previously part of this change, is now covered by #4256
Preview | Diff