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

Name Change - Oriental Tree #20545

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

courierbravo
Copy link
Contributor

Changes the name of the "potted oriental tree" to "potted cherry blossom tree", along with changing the description.

@github-actions github-actions bot added the 🗺️ Mapping - Away Ship/Away Site The PR touches away ship and/or away site map files. label Mar 2, 2025
@FluffyGhoster
Copy link
Contributor

Why are you changing this tree?

@courierbravo
Copy link
Contributor Author

courierbravo commented Mar 2, 2025

Oriental is a dated word with a lot of historical baggage tied to colonialism. Its reductive broadness to include "all (mainland) Asian cultures" and them being "exotic" has lead to its falling out of use. At least where I'm from, its somewhat offensive to use today. While some people still seem to use the word, its considered enough of an issue that in the US, the federal government voted to ban its use in any official documents (see H.R. 4238 amending its use).

There's just better words to use, at the end of the day.

@courierbravo
Copy link
Contributor Author

!review

Copy link
Contributor

@FluffyGhoster FluffyGhoster left a comment

Choose a reason for hiding this comment

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

It's in common usage to refer to things from the east (at least here) ((and according to merriam webster it seems perfectly fine if it's not used to refer to people)) but icba about a tree name

@courierbravo
Copy link
Contributor Author

I see, fair enough. As far as I can really figure, the use of "oriental" really depends. Like, certain things like plants and animals use oriental as a describing feature. Oriental short hair for cats as an example. The issue use case for oriental where it is offensive seems to be to describe people as oriental (hence the house bill i mentioned) or to describe Asia as "the orient."
Side note, what was just pointed out by a friend of mine after making this PR, another probably better reason to make the change is that its just inaccurate to call a clearly cherry tree based plant an oriental tree. No cherry blossom uses "oriental" in its name, and anything from a palm tree to a pine tree would be applicable under the name "oriental tree."

/obj/structure/flora/pottedplant/orientaltree
name = "potted oriental tree"
desc = "This is a rather oriental style tree. It's flowers are bright pink."
/obj/structure/flora/pottedplant/cherrytree
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
/obj/structure/flora/pottedplant/cherrytree
/obj/structure/flora/pottedplant/orientaltree

Eh, it's fine to keep the path name. No one's gonna see it anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, but nah. the point is to remove it because its inaccurate and concerns regarding its use.

Copy link
Contributor

@hazelrat hazelrat left a comment

Choose a reason for hiding this comment

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

1984

@BotBOREALIS BotBOREALIS added the Waiting Period The PR has the required number of reviews, but is still in the waiting period label Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗺️ Mapping - Away Ship/Away Site The PR touches away ship and/or away site map files. Review Required Waiting Period The PR has the required number of reviews, but is still in the waiting period
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants