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 various typos #692

Merged
merged 2 commits into from
Jan 15, 2025
Merged

Fix various typos #692

merged 2 commits into from
Jan 15, 2025

Conversation

luzpaz
Copy link

@luzpaz luzpaz commented Dec 26, 2024

Found via codespell -q 3 -L te,ridiculus,varius

There are several typos I didn't include that perhaps at a later date can be fixed.

./demo/images/material_icons_license.txt:2: commerical ==> commercial
./src/DockAreaTitleBar.cpp:100: overlayed ==> overlaid
./src/DockOverlay.cpp:915: Offest ==> Offset
./src/DockOverlay.cpp:917: Offest ==> Offset

Found via `codespell -q 3 -L te,ridiculus,varius`

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Comment on lines -382 to +385
if (CDockManager::testAutoHideConfigFlag(CDockManager::AutoHideHasCloseButton))
{
if (CDockManager::testAutoHideConfigFlag(CDockManager::AutoHideHasCloseButton))
{
TitleBar->button(TitleBarButtonClose)->setEnabled(
_this->features().testFlag(CDockWidget::DockWidgetClosable));
_this->features().testFlag(CDockWidget::DockWidgetClosable));
Copy link
Author

Choose a reason for hiding this comment

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

github auto-fixed the ^M line end here.

@githubuser0xFFFF
Copy link
Owner

Tank you very much for providing this pull request. I will merge this now.

@githubuser0xFFFF githubuser0xFFFF merged commit a16d17a into githubuser0xFFFF:master Jan 15, 2025
1 check passed
@luzpaz luzpaz deleted the typos branch January 15, 2025 11:13
@luzpaz
Copy link
Author

luzpaz commented Jan 15, 2025

Any feedback on the typos I didn't include? Any interest ?

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

2 participants