-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[docs] Grammar fix for global class names docs #16778
[docs] Grammar fix for global class names docs #16778
Conversation
No bundle size changes comparing 3f9fa1f...d88bc8e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Josh!
|
||
- With a rule name of the [\`classes\` object prop](/customization/components/#overriding-styles-with-classes). | ||
- With a [global class name](/customization/components/#overriding-styles-with-global-class-names). | ||
- With a theme and an [\`overrides\` property](/customization/globals/#css). | ||
|
||
If it's not enough, you can find the [implementation of the component](${SOURCE_CODE_ROOT_URL}${normalizePath( | ||
If that's not sufficient, you can check the [implementation of the component](${SOURCE_CODE_ROOT_URL}${normalizePath( | ||
reactAPI.filename, | ||
)}) for more detail. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
)}) for more detail. | |
)}) for more details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to depend on the context. Do we want to focus on the quality or the quantity of extra information: https://english.stackexchange.com/questions/123672/detail-or-details-read-on-for-more-detail-details?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note, we were using "detail" in the past, highlighting the quality aspect. We are also using "detail" when the ref is attached to an imperative handle.
Reference: #16770 (review)
cc @mbrookes