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 basic spelling check #16

Closed
mstachniuk opened this issue Jan 20, 2015 · 14 comments
Closed

Add basic spelling check #16

mstachniuk opened this issue Jan 20, 2015 · 14 comments

Comments

@mstachniuk
Copy link

No description provided.

@sebbrudzinski sebbrudzinski changed the title Pleas add spelling check Add spelling check Apr 2, 2015
@angelabp
Copy link

maybe an idea to implement http://languagetool.org/ ?

@sebbrudzinski
Copy link
Owner

Thanks for the link @angelabp It's worth investigating that tool.

@WhiteHsu
Copy link
Collaborator

@sebbrudzinski
I'm interested in this, but does anyone has experience with https://languagetool.org/ previously? Or is there any other recommended way to implement this feature?

@sebbrudzinski
Copy link
Owner

Hello @WhiteHsu
We have no previous experience with LanguageTool. The first step in this ticket would be to familiarize with it and investigae whether we can use and if it helps us. Of course we can use any other tool/library as well.

@WhiteHsu
Copy link
Collaborator

I have some fast investigation and find there're some tools mentioned in internet discussions. I list them with information I know so far:

In summary, my idea is to try Language Tool firstly. How about your suggestions? :-)

@sebbrudzinski
Copy link
Owner

Thanks for this investigation @WhiteHsu It seems like all we need at this point is Java API. I think giving Language Tool a shoot first is a good idea.

@WhiteHsu
Copy link
Collaborator

Sorry for late update because being too busy previously. I'll continue my development on this workitem in these days.

@WhiteHsu
Copy link
Collaborator

@sebbrudzinski
Because I'm not very familiar with NetBeans IDE, I encounter several questions and would like to ask your suggestions:

  1. In order to adopt Language Tool, we need to include its JAR library files. However, under the project file structure, I don't find a folder named "libs" or similar ones, so where do you suggest to put these JAR files under the project "OpenLaTeXStudio"?
    default

  2. After copying the JAR files, the next I think we should add them into project. I've followed instructions based on some webpages as this one:
    http://stackoverflow.com/questions/1975973/how-to-use-jar-files-in-netbeans

The steps are: Tool -> Libraries -> New Libraries -> Add JAR/Folder:
default

However, after these steps, I cannot import the packages of Language Tool still. I guess there must be something missed among my steps. So may you help instruct the correct process to add library into project in NetBeans?

  1. In addition to add library files, I guess we should also update pom.xml, correct? But why I find many pom.xml in my built project as below? Which is the one we should update?
    default

Sorry for these questions. Please kindly suggest and feedback. Thank you very much.

@sebbrudzinski
Copy link
Owner

@WhiteHsu We use Maven, that handles the dependencies and external libraries. LanguageTool explains how to add their library to an existing Maven project here: http://wiki.languagetool.org/java-api (it should be as simple as adding a few lines in pom.xml)

@WhiteHsu
Copy link
Collaborator

WhiteHsu commented Feb 3, 2017

@sebbrudzinski
I've checked in updates for this new feature. Here're some points about this feature:

  • For now there's a new green icon as screen below:
    default

  • After clicking this icon, it'll execute spell check and highlight incorrect words:
    default

  • Then you can click the green icon again to clear highlights:
    default

  • This "spell check" can be also triggered by clicking the option in the menu:
    default

Please help review and provide your valuable feedback. Any advice will be very appreciated. Thank you very much. :-)

@WhiteHsu
Copy link
Collaborator

@sebbrudzinski
Is there anything able to improve or enhance? Please kindly help review and provide your valuable feedback on my check-in. Thank you very much. :-)

@sebbrudzinski
Copy link
Owner

@WhiteHsu sorry for delay. Thanks for the pull request, I've reviewed it now. Overall this works, but we need to improve several things in the code.

@WhiteHsu
Copy link
Collaborator

@sebbrudzinski
Thanks for your review comments. I've updated codes based on your review comments. Please kindly help check again and provide your feedback. Thank you very much. :-)

@sebbrudzinski sebbrudzinski changed the title Add spelling check Add basic spelling check Feb 27, 2017
@sebbrudzinski sebbrudzinski added this to the 1.5 milestone Feb 27, 2017
@sebbrudzinski
Copy link
Owner

Fixed by @WhiteHsu New tickets will be created to expand and improve the current spell check

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

No branches or pull requests

4 participants