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

chore: Remove VNU exclusions for accordion #902

Merged
merged 1 commit into from
Oct 1, 2018

Conversation

nschonni
Copy link
Contributor

Last cleanup for #815

@mcking65
Copy link
Contributor

Dang, @nschonni, I merge the accordion changes and you have this ready in the same minute! Amazing! Are you a bot?? LGTM, but waiting for tests to complete.

@nschonni
Copy link
Contributor Author

ha, I was actually testing it this morning but realized that the fixes hadn't merged in yet. I just had to push this when I saw you landed them 😄

@mcking65 mcking65 merged commit c82044e into w3c:master Oct 1, 2018
@nschonni nschonni deleted the vnu-accordion branch October 1, 2018 18:53
michael-n-cooper pushed a commit that referenced this pull request Oct 1, 2018
VNU config:  Remove exclusions for accordion (pull #902)

For issue #815, remove the exclusions in the VNU configuration that permitted accordion-related roles on `dl`, `dt`, and `dt`.
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.

2 participants