Skip to content

Conversation

@Leonabcd123
Copy link
Contributor

@Leonabcd123 Leonabcd123 commented Nov 2, 2025

Description

Added a "both" option to indicate typos that keeps the replace functionality, and makes the below functionality show the correct instead of the incorrect letters. We just check whether the mode is both when deciding whether to pass input to createHintsHtml or to pass currentWord. All functionality of replace and below is kept by just adding or operators to check whether indicateTypos is both OR replace/below.

Implementing #7024

@monkeytypegeorge monkeytypegeorge added frontend User interface or web stuff packages Changes in local packages labels Nov 2, 2025
@Miodec
Copy link
Member

Miodec commented Nov 12, 2025

I think i would rename all to both.

@Miodec
Copy link
Member

Miodec commented Nov 12, 2025

And yes, updateHintsPosition also needs to be changed.

@Miodec Miodec added the waiting for update Pull requests or issues that require changes/comments before continuing label Nov 12, 2025
@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Nov 12, 2025
@Leonabcd123 Leonabcd123 changed the title feat: Add all option to indicate typos (@Leonabcd123) feat: Add both option to indicate typos (@Leonabcd123) Nov 12, 2025
@Leonabcd123 Leonabcd123 changed the title feat: Add both option to indicate typos (@Leonabcd123) feat(test): Add both option to indicate typos (@Leonabcd123) Nov 13, 2025
@Miodec Miodec changed the title feat(test): Add both option to indicate typos (@Leonabcd123) feat(test): add indicate typos: both (@Leonabcd123) Nov 13, 2025
@Miodec Miodec merged commit 13b75f4 into monkeytypegame:master Nov 15, 2025
17 checks passed
@Miodec
Copy link
Member

Miodec commented Nov 15, 2025

Thanks

@Leonabcd123 Leonabcd123 deleted the add-correct-below-option branch November 15, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff packages Changes in local packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants