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

Fixes minor grammatical errors #9336

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tapaswenipathak
Copy link

  1. Removes spaces at end of the lines.
  2. Appropriately replaces doxygen with Doxygen.
  3. Replaces doxywizard with Doxywizard.
  4. Fixes a few other grammatical errors.

Copy link
Collaborator

@albert-github albert-github left a comment

Choose a reason for hiding this comment

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

Disclaimer I'm not a native English speaker

  • the word doxygen should be written wits a lower case d, see also the logo on the cover page of in the doxygen PDF manual
  • the word doxywizard should also be written with a lowercase d

I see on some places that the word doxygen, inside a sentence, is written with an upper case d, this should, still, be corrected. At the beginning of a sentence I think it might be OK to have an upper case d. (@doxygen what do you say).

Comment on lines 59 to +60
generate default style sheet, header, and footer files that you can edit
afterwards:
afterward:
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think that afterwards is more appropriate here.

Comment on lines +20 to 21
argument, otherwise, the default settings will be used as a starting point.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure that the comma after the word otherwise is necessary.

Comment on lines +113 to +114
the Qt documentation. This is done independently of where this documentation
is located!
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think independent is better here.

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.

None yet

3 participants