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

Fix many spelling errors #3244

Merged
merged 6 commits into from
Jun 1, 2021
Merged

Fix many spelling errors #3244

merged 6 commits into from
Jun 1, 2021

Conversation

IRDonch
Copy link

@IRDonch IRDonch commented May 27, 2021

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

  • I submit my changes into the develop branch
  • [ ] I have added description of my changes into CHANGELOG file seems too minor to warrant changelog inclusion
  • [ ] I have updated the documentation accordingly N/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)
  • I have increased versions of npm packages if it is necessary (cvat-canvas,
    cvat-core, cvat-data and cvat-ui)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

zhiltsov-max
zhiltsov-max previously approved these changes May 27, 2021
@@ -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 |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| `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) |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| `Ctrl+Shift+R` | Change image angle (subtract 90 degrees) |
| `Ctrl+Shift+R` | Change image angle (subtract 90 degrees) |

@nmanovic
Copy link
Contributor

@IRDonch , could you please look at linter errors? Looks like in the past we committed these files with some problems.

Copy link
Contributor

@nmanovic nmanovic left a comment

Choose a reason for hiding this comment

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

LGTM

@nmanovic nmanovic merged commit d15a319 into cvat-ai:develop Jun 1, 2021
@IRDonch IRDonch deleted the spellcheck branch September 9, 2022 22:20
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.

3 participants