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

Add A11y for auditing accessibility [PLAT-7883] #4

Closed
wants to merge 1 commit into from
Closed

Conversation

mudnug
Copy link

@mudnug mudnug commented Nov 30, 2016

No description provided.

@mudnug
Copy link
Author

mudnug commented Nov 30, 2016

@billpull, @chriskrycho, @gshackles ... would one of you like to review and merge this?

@mudnug mudnug changed the title Add A11y for auditing accessibility Add A11y for auditing accessibility [PLAT-7883] Nov 30, 2016
@gshackles
Copy link
Member

Can we just make these changes right inside our site Gulpfile(s) for now instead of pushing it into this package? We can abstract things into here down the line as needed.

@mudnug
Copy link
Author

mudnug commented Nov 30, 2016

Sure, that's not a problem. I thought doing so would be rejected, though. I haven't seen that done before

@gshackles
Copy link
Member

Yeah that's fine for stuff like this, especially while we prove out what's actually needed. This is mostly just here to make it easy for projects to access commonly used things, but there's no reason projects can't layer in things of their own too

@gshackles gshackles closed this Nov 30, 2016
@tansongyang tansongyang deleted the Add-A11y branch May 21, 2019 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants