-
Notifications
You must be signed in to change notification settings - Fork 50
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
[BLOG] Improving Accessibility in JupyterLab #537
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
👋
|
Great work, @kulsoomzahra! I made some suggestions. Please ask if you have any questions about any suggestion that I made. 😊 |
Please let me know if there's a need to squash these commits! Thank you |
Also, I'm not sure why the gifs are not showing up in the preview. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kulsoomzahra thanks for addressing the review comments!
Just a few more things to do:
- There were several suggestions I made that you did not accept. Could you either accept them or explain why you're not accepting them?
- Could you rename
hero-image.png
to something else, like accessibility+jupyter.png? (Don't forget to update the path once you change the image name.) I'm afraid it's a little confusing to call it hero image but then not to use it as the hero image. - I made some new suggestions for the title and description. What do you think?
4578f7e
to
ffd1e19
Compare
@gabalafou thanks for reviewing again! I missed out some suggestions, but hopefully it's good now. The only concern I have right now is
|
I'll look into it and get back to you. |
alt: 'Accessibility and JupyterLab logos' | ||
hero: | ||
imageSrc: /posts/improving-accessibility-in-jupyterlab/labsbg.png | ||
imageAlt: 'Accessibility and JupyterLab logos with Quansight labs graphical elements' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
imageAlt: 'Accessibility and JupyterLab logos with Quansight labs graphical elements' | |
imageAlt: 'Jupyter Accessibility' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kulsoomzahra! This is looking great! I have just one last change request. The name of the markdown file serves as the slug for the URL, which means that your blog post would be at the URL: https://labs.quansight.org/blog/improving-accessibility-in-jupyterlab. I would prefer not to use that slug because it's so generic, and I think we may someday want to use that slug for a different blog post.
So would you mind changing the filename of your post to something like zoom-a11y-jupyterlab.md
?
@gabalafou thanks for looping in! Let me know if everything's good. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. update the publish date to this Monday and let's merge this.
@gabalafou are there any outstanding changes you are waiting for?
--- | ||
title: 'Zoom zoom zoom! Improving Accessibility in JupyterLab' | ||
author: kulsoom-zahra | ||
published: October 31, 2022 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
published: October 31, 2022 | |
published: November 28, 2022 |
@bskinn @gabalafou any idea why this blog appears twice in the Labs Blog page preview? I have never seen this before 🤷♀️ |
Never mind, it's still dated Oct 31. I see it. |
@kulsoomzahra can you please remove the markdown file you are not intending to publish from the PR? |
Sure, on it. |
@kulsoomzahra, please also rename the directory your post images are stored in so that they match the filename of your post's Per @gabalafou's comment, this will also avoid possible naming conflicts with a future post. Thanks! |
LGTM. Thanks @kulsoomzahra! |
Text styling
Non-text contents