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

Add spell check feature back in OLW #130

Closed
lon3wolf opened this issue Dec 14, 2015 · 8 comments
Closed

Add spell check feature back in OLW #130

lon3wolf opened this issue Dec 14, 2015 · 8 comments

Comments

@lon3wolf
Copy link

@lon3wolf lon3wolf commented Dec 14, 2015

We can use the built-in API provided by windows Spell check API!

@robertjm
Copy link

@robertjm robertjm commented Jan 7, 2016

This is a big issue IMHO. Don't know about others. But, I relied on the Live Writer spell check quite a bit even though I consider myself a decent speller.

@aabelro
Copy link

@aabelro aabelro commented Jan 7, 2016

+1
Currently, I copy the text to LW to do the spell checking, then, if I made a correction, I copy it back to OLW. We need to spell check everything before pushing it out there.

@lon3wolf
Copy link
Author

@lon3wolf lon3wolf commented Jan 7, 2016

A pull request #213 already exists to fix this.
Once a decision is made to merge and release it, issue should be fixed

willduff added a commit that referenced this issue Jan 14, 2016
Issue #130 - Add spell check feature back in OLW
@willduff
Copy link
Member

@willduff willduff commented Jan 14, 2016

Huge thanks to @bbowyersmyth for fixing this issue! Spell checking will be enabled in the next official Open Live Writer release.

@willduff willduff closed this Jan 14, 2016
@robertjm
Copy link

@robertjm robertjm commented Jan 14, 2016

Thanks @bbowyersmyth !! How often do official releases get put out?

@lextm
Copy link
Contributor

@lextm lextm commented Feb 26, 2016

Should make it very clear that spell check feature is only supported on Windows 8 and above, as "the Spell Checking API is available beginning with the Windows 8 or Windows Server 2012".

@ScottIsAFool
Copy link
Member

@ScottIsAFool ScottIsAFool commented Feb 26, 2016

@lextm you mean like we did in the blog post detailing the release? http://openlivewriter.org/news/release/2016/02/13/version-0-dot-6-now-available.html

@lextm
Copy link
Contributor

@lextm lextm commented Feb 26, 2016

@ScottIsAFool no Scott. I don't mean that. The release notes look great though.

Many threads/issues on GitHub on the other hand, only contains partial information, which is my concern. For example, if anyone comes to this thread, he/she might not get the key information like which OS is supported by spell check feature, and then he/she will be surprised to see this feature is disabled (on Windows Vista or Windows 7 as #209 revealed).

Even googling "spell check Open live writer" won't lead us to the 0.6.0.0 release notes easily.

I believe we need a better way to tell the users. Maybe a pull request to enhance the spell check feature, so that it explicitly tells the users on Windows Vista/7 that they should upgrade.

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

Successfully merging a pull request may close this issue.

None yet
7 participants
You can’t perform that action at this time.