Skip to content

Conversation

kasperisager
Copy link
Contributor

This pull request adds a note with an explanation of the expectation in order to address concerns from the TF that the expectation was a little difficult to understand: w3c/wcag-act#445. As I really don't see how we can reasonably shorten the expectation, a note seemed more appropriate. An existing test case has also been adjusted slightly and a new test case added.

I'm guessing this requires a 2 week final call.

jeeyyy
jeeyyy previously approved these changes May 7, 2020
@WilcoFiers WilcoFiers dismissed jeeyyy’s stale review May 9, 2020 11:23

Was already approved by one person from Deque

…er/act-rules.github.io into css-orientation-lock-feedback

* 'css-orientation-lock-feedback' of github.com:kasperisager/act-rules.github.io:
  aria-props: add a note on HTML validation with aria-current (#1295)
  Semantic role break circularity (#1290)
  id-value-unique: add empty id inapplicable example (#1264)
  audio-text-alt: inapplicable example linking to audio (#1265)
  Make rules consistent with WAI-ARIA 1.2 (#1266)
@kasperisager kasperisager requested a review from Jym77 June 2, 2020 08:30
@kasperisager kasperisager added Review call 2 weeks Call for review for new rules and big changes and removed reviewers wanted labels Jun 2, 2020
@kasperisager
Copy link
Contributor Author

This pull request will be in final call until June 16.

@daniel-montalvo daniel-montalvo removed their request for review June 3, 2020 11:01
Copy link
Collaborator

@EmmaJP EmmaJP left a comment

Choose a reason for hiding this comment

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

Minor grammar tweaks suggested. Otherwise seems fine.

@kasperisager kasperisager merged commit 66f3dc3 into act-rules:develop Jun 23, 2020
@kasperisager kasperisager deleted the css-orientation-lock-feedback branch June 23, 2020 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review call 2 weeks Call for review for new rules and big changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants