Skip to content

Conversation

@SSteve
Copy link

@SSteve SSteve commented Oct 21, 2025

PR summary

Minor grammar corrections in existing documentation.

PR checklist

- Use "its" (the possessive form of "it") instead of "it's" (the contraction of "it is").
- Change plurality of verb to match compound subject joined by "or". The verb should agree with the part of the subject closest to it. See https://editorsmanual.com/articles/compound-subject-singular-or-plural/.
@github-actions github-actions bot added the Documentation: user guide files in galleries/users_explain or doc/users label Oct 21, 2025
We've decided not to include this as a figure method because this is only one
way of defining empty, and checking the above is only rarely necessary.
Usually the user or program handling the figure know if they have added
Usually the user or program handling the figure knows if it has added
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Usually the user or program handling the figure knows if it has added
Usually the user (or program) handling the figure knows if they (it) has added

I think the issue here is that user and program get different pronouns and casting to passive voice won't work b/c the ownership is what's important.

Copy link
Author

Choose a reason for hiding this comment

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

If we parenthesize "or program" and "it", we should change "has" to "have" in order to match the pronoun "they". (Sorry I don't know how to modify a suggested change directly.)

Copy link
Member

Choose a reason for hiding this comment

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

Couldn't we simply write

Suggested change
Usually the user or program handling the figure knows if it has added
Usually the user or program handling the figure know if they have added

?

Copy link
Member

Choose a reason for hiding this comment

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

It's a bad combination of count/program/conjugation agreement.

A user knows what changes they have made and a program knows what changes it has made.

Copy link
Member

Choose a reason for hiding this comment

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

Ask an LLM. They are good at this. 😃 - I leave this to whoever cares because there’s not the one correct answer, and I as non-native speaker are not particularly qualified to judge.

Restructure sentence to avoid subject/pronoun/verb disagreement.
way of defining empty, and checking the above is only rarely necessary.
Usually the user or program handling the figure know if they have added
something to the figure.
Whether or not something has been added to the figure is usually defined
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Whether or not something has been added to the figure is usually defined
Whether or not something has been added to the figure is usually defined

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation: user guide files in galleries/users_explain or doc/users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants