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

Fix name capitalization #630

Merged
merged 1 commit into from
Nov 24, 2024
Merged

Fix name capitalization #630

merged 1 commit into from
Nov 24, 2024

Conversation

karlhorky
Copy link
Contributor

Hi, thanks for Oxipng!

I saw further down in the readme that Oxipng was being used in the middle of a sentence with a capital O, so I adjusted the other lowercase usages (other than the usages in backticks, which may instead refer to the command line program itself) from "oxipng" to also use "Oxipng"

@ace-dent
Copy link

ace-dent commented Jul 1, 2024

Need to check in with @shssoichiro what the naming intention is! :-)

@AlexTMjugador
Copy link
Collaborator

Oddly enough, I tend to capitalize OxiPNG like that, not as Oxipng or oxipng... 😂

@andrews05
Copy link
Collaborator

@AlexTMjugador Personally, I think that makes it look too similar to OptiPNG 😆

@ace-dent
Copy link

ace-dent commented Jul 1, 2024

Not sure if it's open to voting(!?)... but my preference is oxipng and capitalised at the start of a sentence.

@karlhorky
Copy link
Contributor Author

karlhorky commented Jul 1, 2024

oxipng and capitalised at the start of a sentence.

Usually mixing like this will lead to style confusion - often it's better to have a single, strong capitalization style for a brand name and stick with it (regardless of where the word is used in the sentence).

However, that's not to say that the brand name for the project / ecosystem can't be styled differently than the brand name for the command line interface, for example:

  1. Oxipng (capital O, everything else lowercase) for the brand name for the project / ecosystem / anything else
  2. oxipng (all lowercase, enclosed in backticks) for the brand name for the command line interface program

@AlexTMjugador
Copy link
Collaborator

AlexTMjugador commented Nov 24, 2024

I'm still unsure about the preferred capitalization for the project, but I think we can all agree that maintaining consistency within a single document makes it look tidier. Therefore, I'm merging this PR as-is. At the moment, I do not intend this PR to imply that a given capitalization style should be preferred over others, though.

Thanks for the PR! ❤️

@AlexTMjugador AlexTMjugador merged commit 490a469 into shssoichiro:master Nov 24, 2024
12 checks passed
@karlhorky
Copy link
Contributor Author

@AlexTMjugador thanks for the consideration, review and merge!

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

Successfully merging this pull request may close these issues.

4 participants