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

Style: Change esthetic -> aesthetic #93977

Conversation

ericrallen
Copy link
Contributor

This PR ensures consistent spelling for aesthetic across the in-editor docs (and also a few in-code comments where it is written), preferring the more widely-used "ae" spelling.

@ericrallen ericrallen requested review from a team as code owners July 5, 2024 16:23
Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

This is not correct, esthetic is the US English spelling, aesthetic is generally the UK one, and we use the US spelling when it differs

Source: https://en.wiktionary.org/wiki/esthetic#English

@ericrallen
Copy link
Contributor Author

ericrallen commented Jul 5, 2024

Happy to change it to the e if that's really preferred, but I don't think most people actually use the e spelling unless they are referring to skincare, where esthetic and esthetician are popular spellings.

It also seems pretty rare to find anything written about video games, art, or design that doesn't use the ae spelling.

Even as a US-born, native English speaker, I had to Google "estethic" based on your response and this English StackExchange answer has some interesting deep dives into this: https://english.stackexchange.com/q/173585

I just noticed it spelled differently in two places while looking at some in-editor tooltips.

Edit to Add: Just as a note, the Godot Docs only use the ae spelling: https://github.com/search?q=repo%3Agodotengine%2Fgodot-docs%20esthetic&type=code

@AThousandShips
Copy link
Member

I'd say to leave it be, it's a minor difference either way

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

We can keep this for discussion, I don't think this change is worth it as it's a different (yet standard) spelling more US specific which fits our guidelines, though I don't think we should change aesthetic to esthetic either

If you still want this change please fix the translation files, they should not be manually edited, and after that I'll remove the changes requested and we'll have this as up for discussion

@AThousandShips AThousandShips added this to the 4.x milestone Jul 5, 2024
@LinuxUserGD
Copy link
Contributor

This is not correct, esthetic is the US English spelling, aesthetic is generally the UK one, and we use the US spelling when it differs

Source: https://en.wiktionary.org/wiki/esthetic#English

See https://www.merriam-webster.com/dictionary/aesthetic (comment: https://forum.wordreference.com/threads/aesthetic-esthetic.1502784/#post-7604976).
Both variants seem to be equally common in US English spelling.

@ericrallen ericrallen force-pushed the feature/consistent-aesthetic-spelling branch from 0c6576e to e03fb30 Compare July 5, 2024 20:03
@ericrallen ericrallen requested a review from AThousandShips July 5, 2024 20:05
@ericrallen
Copy link
Contributor Author

I removed the changes to the translation files. Sorry for the confusion on that.

If this is too minor a change, feel free to close it - I know it's not actually that important.

I just noticed discrepancies in how the word was spelled between the official docs and then in a few places in the in-editor tooltips and wanted to make it consistent - and aesthetic is the far more common spelling, especially when referring to visual arts.

@Mickeon
Copy link
Contributor

Mickeon commented Jul 5, 2024

There's actually a pretty fair debate on how people expect this word to be spelled as, so keep this PR open. At most it will just not ever be merged.
Note that it's a bit difficult to justify these documentation fixes unless they are typos or plainly inaccurate, because it causes the corresponding translations to become "outdated" even with minor changes.

It's not as clear-cut case of "esthetic is the en_us way of saying it" because both are completely fine even in US English. Now it's a matter of questioning how non-native speakers expect it to be.
One of the lines this PR fixes was even written by me as "aesthetic" originally, before @AThousandShips suggested to change it.

@akien-mga
Copy link
Member

It's anecdotal, but as a non native speaker "aesthetic" is a lot more natural to me.

codespell had the replacement in their en_GB to en_US dictionary for a bit, but removed it as it was indeed pointed out that aesthetic is valid in en_US.

I think we can just pick one and keep it consistent, I don't think this would rub people off the wrong way. Since aesthetic is ok in en_US, I'd say let's be nice to en_GB and international speakers by picking the "ae" variant.

Copy link
Contributor

@Mickeon Mickeon left a comment

Choose a reason for hiding this comment

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

Non-native speaker and it's my own preference too.

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

Don't have a preference either way but this standardized things

Copy link
Contributor

@marcospb19 marcospb19 left a comment

Choose a reason for hiding this comment

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

non-native here too, I didn't know "esthetic" existed and meant "aesthetic", I'm familiar with the latter

Copy link
Member

@aaronfranke aaronfranke left a comment

Choose a reason for hiding this comment

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

I'm from the US, native English speaker, and I've always seen "aesthetic", never any other version. The existence of "esthetic" is brand new information to me. I say let's go with the correct and standard "aesthetic".

@AThousandShips AThousandShips changed the title fix: change esthetic -> aesthetic Change esthetic -> aesthetic Jul 7, 2024
@mrjustaguy
Copy link
Contributor

As a non native speaker, I've only ever heard of aesthetic, and at first esthetic seemed to me like a typo

@akien-mga akien-mga modified the milestones: 4.x, 4.3 Jul 7, 2024
@akien-mga akien-mga changed the title Change esthetic -> aesthetic Style: Change esthetic -> aesthetic Jul 7, 2024
@akien-mga akien-mga merged commit e05ac79 into godotengine:master Jul 7, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

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.

9 participants