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 some notes to the ablist file #51

Merged
merged 8 commits into from Oct 7, 2018
Merged

Conversation

agans
Copy link
Contributor

@agans agans commented Oct 6, 2018

I used the National Center on Disability and Journalism's style guide recommendations to add some additional notes for the ableism script. (http://ncdj.org/style-guide/)

in response to this issue in the alex repo: get-alex/alex#219

-I used the sentence "Only use terms describing mental illness when referring to a professionally diagnosed medical condition." to try to reflect the idea that we shouldn't be using medical terms as an insults or pejorative descriptors.

-I separated out the suggestions on schizophrenia and bipolar disorder, it didn't seem quite right to suggest that "schizophrenic" could be replaced by "person with bipolar disorder" for example.

-I added a note that addiction is considered a disease, to underscore that it is not to be discussed in a pejorative sense.

-I added notes clarifying why not to use language like "suffering from"

@codecov-io

This comment has been minimized.

Copy link
Member

@wooorm wooorm left a comment

Choose a reason for hiding this comment

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

Wow thank you so much @agans! This is amazing. One Q though, that may be a typo?

@@ -32,13 +34,19 @@
considerate:
- people with intellectual disabilities
inconsiderate:
- intellectually disabled people
- sabled people
Copy link
Member

Choose a reason for hiding this comment

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

Should this change be here or is it a typo? Is “sabled people” a thing? 🤔

script/ablist.yml Show resolved Hide resolved
@agans
Copy link
Contributor Author

agans commented Oct 7, 2018

@wooorm Thanks for the great catch on that typo! Fixed it (and found another). Let me know if you see anything else.

I will take a look at some of the other scripts and see if I can do some research + add some notes for those as well. Would also love to dig more into the project and contribute to the code - if there is anything you know of that is needed or could suggest I look into that would be great.

@wooorm wooorm merged commit 84139cf into retextjs:master Oct 7, 2018
@wooorm
Copy link
Member

wooorm commented Oct 7, 2018

Awesome, this is great work @agans. They are super useful addition! And congrats on landing your first contribution here 🎉

For your question, the condition’s proposed in get-alex/alex#210 also need some code changes. It’s not very easy though, but maybe you’d like to try it out! 💪

@agans agans deleted the ag-ablist-notes branch October 7, 2018 16:11
@wooorm wooorm added ⛵️ status/released 🗄 area/interface This affects the public interface 🦋 type/enhancement This is great to have 🧒 semver/minor This is backwards-compatible change labels Aug 13, 2019
@wooorm wooorm added the 💪 phase/solved Post is done label Jul 18, 2021
riley-martine pushed a commit to riley-martine/retext-equality that referenced this pull request Dec 7, 2023
Related to get-alex/alex#219

Closes retextjsGH-51.

Reviewed-by: Titus Wormer <tituswormer@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗄 area/interface This affects the public interface 💪 phase/solved Post is done 🧒 semver/minor This is backwards-compatible change 🦋 type/enhancement This is great to have
Development

Successfully merging this pull request may close these issues.

None yet

3 participants