jbt / markdown-editor

Live (Github-flavored) Markdown Editor
http://jbt.github.com/markdown-editor
ISC License
2.82k stars 643 forks source link

Cannot disable spell checking? #96

Closed iago-lito closed 5 years ago

iago-lito commented 5 years ago

I guess it's related to this post in #38 that hasn't lead to the opening of a new issue yet. I can see no option to disable the spelch.. skellpech.. the checkspeller.. which is annoying if we use custom words or non-english languages:

spellchecker

How do I disable it?

This is it. Thank you for support anyway. This online editor is a nugget :)

aero31aero commented 5 years ago

Hey, it's a pretty easy fix, I'm thinking of doing it soon but I got busy with college. I'll try to get on this in the next couple of days.

I'm thinking of making this another toggle button on top.

On Tue, Oct 23, 2018, 6:30 PM Iago-lito notifications@github.com wrote:

I guess it's related to this post https://github.com/jbt/markdown-editor/issues/38#issuecomment-424627651 in #38 https://github.com/jbt/markdown-editor/issues/38 that hasn't lead to the opening of a new issue yet. I can see no option to disable the spelch.. skellpech.. the checkspeller.. which is annoying if we use custom words or non-english languages:

[image: spellchecker] https://user-images.githubusercontent.com/10091387/47361994-a802cf00-d6d3-11e8-9c04-966f165178be.png

How do I disable it?

This is it. Thank you for support anyway. This online editor is a nugget :)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jbt/markdown-editor/issues/96, or mute the thread https://github.com/notifications/unsubscribe-auth/AHqT1rjCfdKZiiHMz7jFI2DsXE2qjx3eks5unxLugaJpZM4X1ggP .

iago-lito commented 5 years ago

Sounds pretty straightforward indeed. No worries about delays, this is pretty low down priority lists :)

aero31aero commented 5 years ago

@iago-lito check it out and tell if the fix works fine for you. I've got saving the preferences in the browser on the map as well so you don't have to always keep toggling them but that'd be a slightly larger refactor.

iago-lito commented 5 years ago

Wow, nice ^ ^ Well, the button is there, but the checker seems always disabled no matter how much I click it. Also, I now get a firefox

This page is asking you to confirm that you want to leave -
data you have entered may not be saved.

popup message whenever I try to leave the page.
Does it work fine on your side ?

aero31aero commented 5 years ago

There seems to be a bug on the side of the spellchecker in that it only computes the spellcheck on a keystroke. So, after a reload to the page, unless I type something, I don't see the spellcheck errors. Making any changes to the text fixes that.

About the second point, see #95. I'm pushing a commit to make those warnings less intrusive.

iago-lito commented 5 years ago

Everything fixed. Cheers, thank you :) And keep up with the good work! I find this website really neat and I regret it's not used more often by my colleagues :)