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

feat(www): add accessibility statement #13190

Merged
merged 6 commits into from
Apr 8, 2019
Merged

feat(www): add accessibility statement #13190

merged 6 commits into from
Apr 8, 2019

Conversation

marcysutton
Copy link
Contributor

@marcysutton marcysutton commented Apr 7, 2019

Description

This PR adds a central place for Gatsby accessibility information to live on the .org site. It's currently linked from the homepage, with a centered text link at the bottom. We should add it to other sections too, but that will require some design work to fit a footer on pages that go full-height. This PR was intended to ship something we can link to, specifically for @amberleyromo's talk at JAMStack conf.

We'll want to update the accessibility statement whenever we have new information to add, particularly around our current status / commitment to WCAG 2.0 (and the last edited date on the statement...perhaps could be automated? seemed like overkill to me.)

EDIT: the homepage link was removed and #13221 was opened to sort out the design / placement.

@marcysutton marcysutton requested a review from a team April 7, 2019 05:38
@marcysutton marcysutton requested a review from a team as a code owner April 7, 2019 05:38
@marcysutton
Copy link
Contributor Author

marcysutton commented Apr 8, 2019

@fk any design input on the impromptu homepage footer? (see #13221)

@DSchau DSchau changed the title Add accessibility statement to homepage feat(www): add accessibility statement to homepage Apr 8, 2019
Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

In the interest of not blocking this PR--could we introduce the document, and then in a separate PR introduce the footer?

The document and content itself looks great--just a little less sure about the footer, particularly the design and content (and I'd defer to @fk on the design, of course!).

docs/accessibility-statement.md Outdated Show resolved Hide resolved
docs/accessibility-statement.md Outdated Show resolved Hide resolved
@marcysutton
Copy link
Contributor Author

In the interest of not blocking this PR--could we introduce the document, and then in a separate PR introduce the footer?

Yes we can. I'll remove the link with a commit and open an issue to sort out placement!

DSchau and others added 2 commits April 8, 2019 09:45
Co-Authored-By: marcysutton <holla@marcysutton.com>
Co-Authored-By: marcysutton <holla@marcysutton.com>
Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

Preemptively approving for when the footer is removed!

Thanks for this Marcy! So glad to see this here!!

@marcysutton marcysutton changed the title feat(www): add accessibility statement to homepage feat(www): add accessibility statement Apr 8, 2019
@marcysutton marcysutton merged commit 8145a59 into master Apr 8, 2019
@marcysutton marcysutton deleted the a11y-statement branch April 8, 2019 17:04
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