Skip to content

[Edit] Accessibility: Add Practicality and Challenges #6579

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

Merged
merged 5 commits into from
May 14, 2025

Conversation

AnamolR
Copy link
Contributor

@AnamolR AnamolR commented Apr 17, 2025

Description

This PR updates the existing Accessibility entry in the uiux/concepts/design-methodologies/terms section to include two new content sections: What Accessibility Looks Like in Practice and Challenges Designers Should Consider.

Markdown file: accessibility.md
The additions enhance the entry by offering real-world implementation guidance and highlighting common accessibility challenges in UX design. These updates strengthen the practical and inclusive design value of the content for learners and practitioners.

Issues Solved

Closes #6603

Type of Change

Editing an existing entry (fixing a typo, bug, issues, etc)

Checklist

  • All writings are my own.
  • My edits follow the Codecademy Docs style guide.
  • My changes generate no new warnings.
  • I have performed a self-review of my own writing and code.
  • I have checked the updated content and corrected any misspellings.
  • I have made corresponding changes to the documentation if needed.
  • I have confirmed my changes are not being pushed from my forked main branch.
  • I have confirmed that I'm pushing from a new branch named after the changes I'm making.
  • I have linked any issues that are relevant to this PR in the Issues Solved section.

@mamtawardhani
Copy link
Collaborator

mamtawardhani commented Apr 17, 2025

Hey @AnamolR, at Codecademy we ask the contributors to raise a PR that traces back to an issue. You can find the issues here. I see that you've raised some PRs that don't really trace back to any issue. In this case, we will have a look at them and let you know within a few days. Please don't close the PRs :)

For now, you can check the open issues, if anything interests you - please comment "claim" on it. Once you have been assigned to that issue, you can raise a PR. Thank you! 😄

For visibility @avdhoottt @Radhika-okhade

@AnamolR
Copy link
Contributor Author

AnamolR commented Apr 17, 2025

Hi @mamtawardhani, thank you for responding 🙂
Would it be helpful if I created issues for each of the PR that I raised? Currently, there's only one issue that's linked to my PR. My previous PRs were merged without an issue being created so that didn't occur to me. Would be happy to do that.
Thank you 🙂

@mamtawardhani
Copy link
Collaborator

Hey @AnamolR, I guess you can create issues, but once let us ask @Radhika-okhade.

@Radhika-okhade please let us know a solution to this? Should @AnamolR be raising issues or should we raise those for him?

@Radhika-okhade
Copy link
Collaborator

Hey! @AnamolR Yes you can create issues for the PR's that you have raised.

@AnamolR
Copy link
Contributor Author

AnamolR commented Apr 17, 2025

Great, will be on it 🙌. Thank you both @mamtawardhani @Radhika-okhade 🙂

@mamtawardhani mamtawardhani self-assigned this Apr 27, 2025
Copy link
Collaborator

@mamtawardhani mamtawardhani left a comment

Choose a reason for hiding this comment

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

Looks good for a second round of review! 🚀

@avdhoottt avdhoottt added status: under review Issue or PR is currently being reviewed and removed status: ready for next review labels May 2, 2025
@avdhoottt avdhoottt self-assigned this May 2, 2025
Copy link
Collaborator

@avdhoottt avdhoottt left a comment

Choose a reason for hiding this comment

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

LGTM!!

@avdhoottt avdhoottt added status: review 2️⃣ completed and removed status: under review Issue or PR is currently being reviewed labels May 14, 2025
@avdhoottt avdhoottt merged commit 226ba57 into Codecademy:main May 14, 2025
7 checks passed
Copy link

👋 @AnamolR
You have contributed to Codecademy Docs, and we would like to know more about you and your experience.
Please take a minute to fill out this four question survey to help us better understand Docs contributions and how we can improve the experience for you and our learners.
Thank you for your help!

🎉 Your contribution(s) can be seen here:

https://www.codecademy.com/resources/docs/uiux/design-methodologies/accessibility

Please note it may take a little while for changes to become visible.
If you're appearing as anonymous and want to be credited, visit the linked accounts page and ensure that your GitHub account is linked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Edit] Accessibility: Add Practicality and Challenges
4 participants