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.

Sign up for GitHub

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

Jump to bottom

Limit the length of characters for current_word #66

Merged
merged 4 commits into from Nov 1, 2020
Merged

Limit the length of characters for current_word #66

merged 4 commits into from Nov 1, 2020

Conversation

jai-dewani
Copy link
Contributor

This PR fixes #59

Tested with python version - 3.8.5

On operating system - Windows 10 version 2004

Mithil467
Mithil467 requested changes Nov 1, 2020
View reviewed changes
mitype/app.py Outdated Show resolved Hide resolved
Mithil467
Mithil467 requested changes Nov 1, 2020
View reviewed changes
Copy link
Owner

@Mithil467 Mithil467 left a comment

Choose a reason for hiding this comment

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

This works great. Just this one change and we are good to merge this!

mitype/app.py Outdated Show resolved Hide resolved
@Mithil467
Copy link
Owner

Thank you so much for your contribution! @jai-dewani 😄

@Mithil467 Mithil467 changed the title Add limit on current_word Limit the length of characters for current_word Nov 1, 2020
@jai-dewani
Copy link
Contributor Author

Thanks for keeping up with my minor shouldn't_have_done_those mistakes :p

@Mithil467 Mithil467 merged commit 8194a37 into Mithil467:master Nov 1, 2020
@jai-dewani jai-dewani deleted the limit_current_word branch November 1, 2020 12:12
@Mithil467 Mithil467 mentioned this pull request Sep 4, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mithil467 Mithil467 Mithil467 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Visual glitch when using backspace to clear text on multiple rows
2 participants
@jai-dewani @Mithil467