gtarawneh / languagetool-sublime

Proof-reading and grammar checker for English, French, German, Polish and 20+ other languages
71 stars 13 forks source link

Not all errors are highlighted when used with Markdown Editing plugin #10

Closed Kristinita closed 7 years ago

Kristinita commented 8 years ago

Summary

Not all errors are highlighted when used with Markdown Editing plugin. This plugin has been downloaded more than 200,000 times, it is included in the Top 100 of plugins, so I think it is necessary to consider its use.

Expected behavior

Highlighting all errors as other syntaxes.

PlainText syntax

In code fragments I also saw typos:

Typos in code

Actual behavior

Highlighting only current error.

MarkdownEditing

Steps to reproduce

I installed a clean Sublime Text without plugins MarkdownEditing and LanguageTool. I open file with .md extension. Ctrl+Shift+PLanguage Tool: Check Text. I chose different syntaxes: Markdown, Markdown GFM and MultiMarkdown — problem still exists.

Environment

Kristinita commented 7 years ago

In Markdown GFM syntax is white selection on white background. But how change color of selection? I try to change different parameters in MarkdownEditor.TmTheme file, but I was not able to change the color. Thanks.

gtarawneh commented 7 years ago

@Kristinita Can you provide a sample markdown file to reproduce this issue?

Kristinita commented 7 years ago

@gtarawneh , https://gist.github.com/a1deddf4c8a7fb84b7435ef3c5ec0996. Stack trace after Alt+↓ in line 24. Thanks.

gtarawneh commented 7 years ago

@Kristinita Is this an intermittent issue or can you reproduce it reliably? in the latter case can you list the exact steps to reproduce (including the full stack trace please)? I've installed MarkdownEditing and ran a Language check on the file https://gist.github.com/a1deddf4c8a7fb84b7435ef3c5ec0996 but I was able to navigate between language problems without any issue.

Thanks for your patience.

Kristinita commented 7 years ago

@gtarawneh, I'm sorry, when I wrote my previous comment, I think, what I wrote comment to issue #12. Previous my comment for issue #12.

Problem in this issue I reproduce on new Sublime Text without plugins and user settings. Steps to reproduce:

  1. Install MarkdownEditing plugin.
  2. Open any markdown file.
  3. Ctrl+Shift+PSetSyntax: Markdown GFM.
  4. Run LanguageTool in opened markdown file.

Thanks.

Kristinita commented 7 years ago

@gtarawneh , any ideas?

gtarawneh commented 7 years ago

@Kristinita Sorry I've had trouble reproducing this issue so had to push it down the queue. I'll get on it as soon as I get some free time.

lf-araujo commented 7 years ago

@gtarawneh I can reproduce it in my machine. I am running subl3 in Manjaro.

Kristinita commented 7 years ago

@facelessuser, can you give the cue, how we can fix these problem? For example, for your package RegReplace results for Markdown GFM syntax are highlight black color:

**Reg Replace**

Selection color ≠ background color. It's good.

Thanks.

facelessuser commented 7 years ago

So, let me get this straight. You are unhappy with the color this plugin highlights with?

If that is true, the author could make the scope they use to highlight configurable. Plugin highlight regions are added by specifying a scope to use to get the color. If that scope is similar to the background color, you will get faint highlights. I took a quick look, and I think this plugin uses the String scope to highlight regions. Does this sound right? I usually expose these kind of things in my settings file when writing plugins. Just because it is a good color on my theme does not mean it will be a good color on all themes.

Hopefully that helps and I interpreted the issue correctly.

Kristinita commented 7 years ago

@gtarawneh ?

gtarawneh commented 7 years ago

Alright, I added a highlight-scope setting to enable users to customize the outline color for detected problem. I've set the default to comment because it seems to work well across different themes, including Markdown GFM but the setting can be changed to comment, constant or any of the region types listed in:

https://www.sublimetext.com/docs/3/scope_naming.html