desktop icon indicating copy to clipboard operation
desktop copied to clipboard

Spell check being overly aggressive

Open alex-taffe opened this issue 4 years ago • 11 comments

I confirm (by marking "x" in the [ ] below: [x]):


Summary The spellchecker is correcting before words are done being typed, for example correcting immediately to I’mmediately. This started when upgrading from 4.4.X to 4.6.X

Environment

  • Operating System: macOS 10.15.7 (19H2)
  • Mattermost Desktop App version: Version 4.6.1 (4.6.1.7401)
  • Mattermost Server version: 5.28.0

Steps to reproduce

  1. Open chat
  2. Type a word that might be a contraction (ex: idk or immediately)
  3. Note the correction to I’dk or I'mmediately

Expected behavior The auto correct respects macOS system behavior and doesn’t autocorrect until the word is done being typed

Observed behavior The auto correct activates after typing the next letter (ex when typing immediately, typing the 2nd m immediately changes the string im to I’m)

Possible fixes

alex-taffe avatar Nov 05 '20 16:11 alex-taffe

Hello! and thanks for reporting. Do you have osx to automatically correct the spelling? https://support.apple.com/en-hk/guide/mac-help/mchlp2299/mac

Willyfrog avatar Nov 05 '20 16:11 Willyfrog

image I do yes, but it still shouldn’t be correcting until after space is pressed

alex-taffe avatar Nov 05 '20 17:11 alex-taffe

For what it’s worth, I have im a a manual replacement to I’m in system preferences. But it doesn’t cause problems in any other app image

alex-taffe avatar Nov 05 '20 17:11 alex-taffe

if you untick the correct spelling automatically option, does it behave properly?

Willyfrog avatar Nov 06 '20 10:11 Willyfrog

That does not cause it behave properly. But if I remove my replace im with I'm rule, it does behave properly, so seems to be related to the replacement

alex-taffe avatar Nov 06 '20 13:11 alex-taffe

But if I remove my replace im with I'm rule, it does behave properly, so seems to be related to the replacement

that seems to hint that the os is the problem not so much the app, not sure how to help you here :/

Willyfrog avatar Nov 09 '20 10:11 Willyfrog

It’s only recent versions of Mattermost where it’s occurring though. It doesn’t occur in older versions, nor any other app on my system. Every other app respects how the behavior is supposed to work

alex-taffe avatar Nov 09 '20 13:11 alex-taffe

is there anything else affecting this replacements? I've added a rule for im to be replaced by I'm and it is suggested but not forced. also if I hit any key other than space the suggestion gets removed. if I hit space it does get replaced.

Willyfrog avatar Nov 10 '20 10:11 Willyfrog

I don’t see anything else that could be causing it, but not sure

alex-taffe avatar Nov 17 '20 18:11 alex-taffe

i've just rechecked in big sur and I can't reproduce this behaviour either.

When I type Im it suggests using I'm, but doesn't replace right away, if I type space it gets corrected, if I type any other letter the suggestion dissappears and doesnt substitute anything. AFAIK this is the expected behaviour.

I'm leaving the issue open in case someone can provide more details, but without being able to reproduce it is hard to think of possible solutions (either by modifying the app or by config)

Willyfrog avatar Nov 18 '20 11:11 Willyfrog

This seems to be an issue with Electron as a whole as Slack is doing the same thing. Filed a bug on that project https://github.com/electron/electron/issues/32101

alex-taffe avatar Dec 03 '21 20:12 alex-taffe

@alex-taffe Is this still an issue in Mattermost? Looking at the upstream issue it looks like Slack has the issue as well.

devinbinnie avatar May 05 '23 15:05 devinbinnie

@devinbinnie I'm honestly not sure, I've moved to a company that uses Slack. It's definitely still an issue in Slack and driving me nuts

alex-taffe avatar May 05 '23 15:05 alex-taffe

Looks like this issue was closed by Electron and doesn't look like it was re-opened. I'm guessing something has been fixed or dealt with elsewhere, so I'm going to close as inactive for now.

Will re-open if Electron opens a new issue or it comes up again.

devinbinnie avatar May 11 '23 20:05 devinbinnie