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: Add spellcheck=false to match decorations #245

Merged
merged 1 commit into from
Nov 9, 2022

Conversation

jonathonherbert
Copy link
Contributor

What does this change?

Disable browser spellchecking in matched text.

Before After
Screenshot 2022-11-07 at 09 31 56 Screenshot 2022-11-07 at 09 32 30

How to test

Running locally, and making sure your browser's spellcheck is on, introduce some words to the sandbox that result in a hit from the spellcheck and Typerighter simultaneously ('Keir Starmer' does the trick for the basic spellcheck in Chrome.) They should match the result above.

@@ -66,7 +66,7 @@ const {
typerighterTelemetryAdapter.matchDecorationClicked(match, document.URL),
requestMatchesOnDocModified: true,
adapter: new TyperighterChunkedAdapter(
"https://checker.typerighter.local.dev-gutools.co.uk"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not strictly related but added as part of this PR – it's much more likely developers are working with the CODE sandbox when developing PM-TR locally, and I'm forever correcting when making PRs!

@jonathonherbert jonathonherbert marked this pull request as ready for review November 7, 2022 09:44
@jonathonherbert jonathonherbert requested a review from a team November 7, 2022 09:47
@jonathonherbert jonathonherbert merged commit b24daa1 into main Nov 9, 2022
@jonathonherbert jonathonherbert deleted the jsh/remove-spellcheck-from-suggestions branch November 9, 2022 10:54
@github-actions
Copy link

github-actions bot commented Nov 9, 2022

🎉 This PR is included in version 6.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants