Tresjs / tres

Declarative ThreeJS using Vue Components
https://tresjs.org
MIT License
1.93k stars 86 forks source link

Minor error in docs- ThisIsPascalCase and thisIsCamelCase #621

Closed jonmatthis closed 3 months ago

jonmatthis commented 3 months ago

Describe the bug

In the 'your first scene' doc, you say the 'TresAmbientLight' is camel case. Technically it is PascalCase 😌

Reproduction

https://docs.tresjs.org/guide/your-first-scene.html

Steps to reproduce

  1. Look at words.
  2. Think of their definitions.
  3. Be super pendantic
  4. Clutch pearls 😱

System Info

No response

Used Package Manager

npm

Code of Conduct

andretchen0 commented 3 months ago

Context

ThisSortOfCasing is called:

If we're going to change the wording to be more specific, I'd personally prefer 'UpperCamelCase' or 'CamelCaps'; 'PascalCase' isn't evocative to me.

jonmatthis commented 3 months ago

Oh, I didn't realize there was inconsistency there! Thanks for clarifying.

I agree - the term PascalCase always stood out as a weirdly non-evocative term amid the lovely pantheon of camelCase, kebab-case, snake-case, and (my personal favorite) SCREAMING_SNAKE_CASE

Like, who th is this Pascal guy? Like the gas science guy? What's he got to do with capitalization?

UpperCamelCase seems like the good choice.

Thanks for your work!

andretchen0 commented 3 months ago

What's he got to do with capitalization?

Origin of 'PascalCase' as a term in computer programming.


Fwiw, I filed a PR changing "CamelCase" to "UpperCamelCase" in the docs.