-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 many spelling errors #3244
Fix many spelling errors #3244
Conversation
@@ -29,13 +29,13 @@ Many UI elements have shortcut hints. Put your pointer to a required element to | |||
| | _Modes_ | | |||
| `N` | Repeat the latest procedure of drawing with the same parameters | | |||
| `M` | Activate or deactivate mode to merging shapes | | |||
| `Alt+M` | Activate or deactivate mode to spliting shapes | | |||
| `Alt+M` | Activate or deactivate mode to splitting shapes | |
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.
| `Alt+M` | Activate or deactivate mode to splitting shapes | | |
| `Alt+M` | Activate or deactivate mode to splitting shapes | |
Adjusted grid
| `G` | Activate or deactivate mode to grouping shapes | | ||
| `Shift+G` | Reset group for selected shapes (in group mode) | | ||
| `Esc` | Cancel any active canvas mode | | ||
| | _Image operations_ | | ||
| `Ctrl+R` | Change image angle (add 90 degrees) | | ||
| `Ctrl+Shift+R` | Change image angle (substract 90 degrees) | | ||
| `Ctrl+Shift+R` | Change image angle (subtract 90 degrees) | |
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.
| `Ctrl+Shift+R` | Change image angle (subtract 90 degrees) | | |
| `Ctrl+Shift+R` | Change image angle (subtract 90 degrees) | |
@IRDonch , could you please look at linter errors? Looks like in the past we committed these files with some problems. |
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.
LGTM
Fix various spelling errors that I found with codespell (and a few that I noticed myself). In a few cases, fix bad grammar, as well.
I left some errors unfixed in the cases where they were a part of the API (primarily ones involving "splitted"), where they were in generated code, or where I didn't fully understand the meaning of the sentence.
Motivation and context
Bad spelling hurts comprehension and looks unprofessional. :-)
How has this been tested?
I ran the
cvat-core
unit tests, but other than that, I'm relying on GitHub CI.Checklist
develop
branch[ ] I have added description of my changes into CHANGELOG fileseems too minor to warrant changelog inclusion[ ] I have updated the documentation accordinglyN/A, although some fixes are to the documentation, as well.[ ] I have added tests to cover my changes[ ] I have linked related issues (read github docs)cvat-core, cvat-data and cvat-ui)
License
Feel free to contact the maintainers if that's a concern.