Clarifying the purpose of correctAutomatically
eengoron opened this issue · 3 comments
Hey @sindresorhus --
First, I just wanted to say thank you for getting spellcheck merged. This is awesome!
I'm currently updating my application to use native spellcheck, and I had a quick question about the correctAutomatically
menuitem.
How is this item supposed to be used? From
electron-context-menu/index.js
Lines 44 to 50 in 1286195
Is there something I'm missing here? If that is the case, would you be open to a PR to hide it behind a showCorrectAutomatically
flag?
Thanks again!
You're right. It's not very useful. It should not have been added. I missed how it worked when reviewing the PR. The PR that added the spellchecking was pretty sloppy, so I'm not surprised I missed something.
See: https://github.com/sindresorhus/electron-context-menu/releases/tag/v2.0.0
Thanks @sindresorhus!