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

Highlight current word #37

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

Conversation

ahnwarez
Copy link

@ahnwarez ahnwarez commented Dec 3, 2023

First, I want to say thank you for making this awesome app. I always recommend it to my students who want improve touch typing.

This PR highlights the current word in white color, but future and past words are in muted color.

The reason for making this PR is that my eyes kinda get lost when I practice longer combinations (higher than ten) and I though maybe adding a visual indication could help other users to stay focused.

This is my first PR so any feedback is welcomed.

@ranelpadon
Copy link
Owner

Sorry, for the late reply. Thanks for this interesting/useful PR! Will test/review asap this week. Cheers

@rdhar
Copy link

rdhar commented Feb 28, 2024

Perhaps revisit?

@jm-janzen
Copy link

Would love to see this change make it in. Literally came to this repo to see if I could contribute exactly this.

@emilkm
Copy link

emilkm commented Aug 4, 2024

I like this contribution. Added it to my fork (https://github.com/emilkm/ngram-type). You can test/use it here https://kesten.nz/ngram-type

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.

5 participants