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

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

Closed
Kristinita opened this issue Jul 23, 2016 · 12 comments
Closed

Comments

@Kristinita
Copy link
Contributor

Kristinita commented Jul 23, 2016

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

  • Operating system and version:
    • [√] Windows 10 Education
  • Sublime Text:
    • Build 3114
@Kristinita
Copy link
Contributor Author

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
Copy link
Owner

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

@Kristinita
Copy link
Contributor Author

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

@gtarawneh
Copy link
Owner

@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
Copy link
Contributor Author

@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
Copy link
Contributor Author

@gtarawneh , any ideas?

@gtarawneh
Copy link
Owner

@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
Copy link

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

@Kristinita
Copy link
Contributor Author

Kristinita commented Jan 9, 2017

@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
Copy link

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
Copy link
Contributor Author

@gtarawneh ?

@gtarawneh
Copy link
Owner

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

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

No branches or pull requests

4 participants