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

impr(read ahead): show words if error in current (@notTamion) #6006

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

Conversation

notTamion
Copy link
Contributor

Description

Makes it easier to recover when not correctly remembering a word

@monkeytypegeorge monkeytypegeorge added frontend User interface or web stuff assets Languages, themes, layouts, etc. labels Nov 4, 2024
@Miodec
Copy link
Member

Miodec commented Nov 6, 2024

The issue i see with this is that if you're typing fast, you will get some very annoying flashing of the words if you make mistakes but keep typing. Also, the words shouldnt be shown if blind mode is enabled.

So yeah, im not sure about it, the flashing could be very annoying.

@Miodec Miodec added the waiting for update Pull requests or issues that require changes/comments before continuing label Nov 6, 2024
@notTamion
Copy link
Contributor Author

Yeah i guess this depends on your typing style. I personally prefer to go back and fix my errors. Will come back to this when im home

@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Nov 6, 2024
@notTamion
Copy link
Contributor Author

Now it only shows the words if you backspace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets Languages, themes, layouts, etc. backend Server stuff frontend User interface or web stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants