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

Typo: Corrected Ethereum.org -> ethereum.org on Design contribution page #12626

Merged
merged 2 commits into from
Apr 8, 2024

Conversation

HiroyukiNaito
Copy link
Contributor

Corrected Ethereum.org -> ethereum.org on Design contribution page

Description

Corrected Ethereum.org -> ethereum.org on Design contribution page

Related Issue

None

@github-actions github-actions bot added the content 🖋️ This involves copy additions or edits label Apr 1, 2024
Copy link

netlify bot commented Apr 1, 2024

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 1b719d3
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/660f7b3cff05f10008ca169f
😎 Deploy Preview https://deploy-preview-12626--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -30,7 +30,7 @@ Provide feedback on our website by:

### <Emoji text=":three:" size={1} /> &nbsp;Find design related issues on the website and report them {#report-design-issues}

Ethereum.org is a fast growing website with many features and content. Some of the UI can easily become obsolete or could be improved. If you encounter any such instance, please report it so that it gets our attention.
ethereum.org is a fast growing website with many features and content. Some of the UI can easily become obsolete or could be improved. If you encounter any such instance, please report it so that it gets our attention.
Copy link
Member

Choose a reason for hiding this comment

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

Hm... Looking at this, we use a split throughout the repo when we start a sentence with the domain/name... about 47 use lowercase, and about 99 capitalize.

Personally think starting a sentence with a lowercase letter always looks awkward and would advocate for using "Etheruem.org" in these cases.

@minimalsm Thoughts here?

Suggested change
ethereum.org is a fast growing website with many features and content. Some of the UI can easily become obsolete or could be improved. If you encounter any such instance, please report it so that it gets our attention.
Ethereum.org is a fast growing website with many features and content. Some of the UI can easily become obsolete or could be improved. If you encounter any such instance, please report it so that it gets our attention.

Copy link
Contributor Author

@HiroyukiNaito HiroyukiNaito Apr 4, 2024

Choose a reason for hiding this comment

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

I thought ethererm.org is started with always lowercase as a proper noun.
Look like this page https://ethereum.org/en/about/.
Similar to iPad and iPhone.
If we use IPhone is very cool looks awkward.

Copy link
Contributor Author

@HiroyukiNaito HiroyukiNaito Apr 4, 2024

Choose a reason for hiding this comment

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

Or maybe it needs to be aligned with Ethereum.org as a proper noun for preventing this situation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I opened the discussion about this.

#12669

Copy link
Member

Choose a reason for hiding this comment

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

Great thanks! We don't usually use GitHub "Discussions" so I'm honestly not sure how much traction it'll get over there.

This isn't something I personally have a strongly held opinion on, was just noting above that we more commonly use "Ethereum.org" over "ethereum.org" when starting a sentence. I tend to prefer consistency above all.

I agree with your point about how "iPhone" is used, not "IPhone", but still feels a little strange I guess, but not a big deal.

cc: @minimalsm Did you have any thoughts on this? We have always tended to use the lowercase domain as the official name, so I wouldn't be against just formalizing that and keeping that pattern even when starting a sentence.

Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Given this PR wasn't intending to set the new standard, and to just clean this one up, happy to bring this in and we can continue to discuss potentially cleaning up other instances of this.

@wackerow wackerow merged commit b319a32 into ethereum:dev Apr 8, 2024
6 checks passed
This was referenced Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants