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

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

Conversation

karlhorky
Copy link

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
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

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