Blog post: Reviewing for accessibility (JLab accessibility 4)

isabela-pf commented 2 years ago

Text styling

[x] The blog is written with plain language (where relevant).
[x] If there are headers, they use the proper header tags in order to do so (with only one level-one header).
[x] All links describe where they link to (for example, check the Quansight labs website).
[x] Any kind of styling that the author uses (for example, bold for emphasis) is consistent throughout the blog.

Non-text contents

[x] All content is represented as text (for example, images need alt text and videos need captions or descriptive transcripts).
[x] If there are emojis, there are not more than three in a row.
[x] Don't use flashing gifs or videos.
[x] If it were to be read as plain text, the blog still makes sense and no information is missing.

Phew! 😓 I'm finally back with another blog post. This one has been in progress for several months, so I'm very happy to get something up for review. This one is replying to many questions I've gotten about what I'm trying out when I review content for accessibility considerations.

I know I've been a part of some of the recent blog posts, so if we want to delay posting this I'm happy to wait until we get some other posts queued up if you'd like.

Things I'd like review on:

I left the All links describe where they link to checkbox empty because I would like review on that. I'm not sure if there's a better way to format them.
I've been trying to figure out a way to organize this in a more digestible way, but I haven't come up with one. If you have thoughts on this or complaints on my current approach, it'd be good to hear. I've thought about flow charts, choose-your-own-adventure things, or just different headers, but I haven't had the brain power to make those ideas make sense. The good news is that I don't think this blog post needs to be read end-to-end to be useful.

Thank you so much!

netlify[bot] commented 2 years ago

Deploy Preview for admiring-northcutt-a5be8c ready!

Name	Link
Latest commit	0be82a19edf592694c52b26750d04e641d9fbcee
Latest deploy log	https://app.netlify.com/sites/admiring-northcutt-a5be8c/deploys/62a380994ddd7600097ee799
Deploy Preview	https://deploy-preview-324--admiring-northcutt-a5be8c.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

isabela-pf commented 2 years ago

@gabalafou thank you for the thorough review! It was very needed for this post in particular.

I think I've addressed almost all of your comments for changes, minus other checklist posts and an example for knowing what you don't know. I can work on addressing these soon.

Please let me know if I didn't successfully address other comments and/or you have new thoughts!

isabela-pf commented 2 years ago

Okay, all addressed! Thanks again for making this ten times better, @gabalafou!

This is ready for more review or merging whenever people in charge want.

isabela-pf commented 2 years ago

@rgommers and/or @trallard I think this is ready for your final review. Please let me know if there's anything you still want me to change.

isabela-pf commented 2 years ago

Thanks, I'll keep that in mind! I never have a deadline for these blogs, so I don't worry about it.

Quansight-Labs / quansight-labs-site