Docstring Highlighting about atom-language-julia HOT 10 CLOSED

These fixes would be great.

Re: improving highlighting generally, I have a PR with comprehensive improvements that I'm pretty excited about, but they're languishing locally until a parsing bug is fixed upstream: atom/first-mate#52. I don't know if anyone has done any work on first-mate -- hopefully it's fixed soon.

from atom-language-julia.

@garborg, really looking forward to your improvements. I think a solid cleanup would be great. Most of what is there right now is just inherited from the textmate bundle. We've made a few changes, but not a general overhaul like I think you are working on.

from atom-language-julia.

@one-more-minute I think you can just put a space between the _ and ) to fix the italic issue. I just tested and it looks like that would fix the docstring not ending issue also.

This also seems like more of a problem in the atom-gfm grammar since it only happens when we include their grammar (i.e. only happens in a docstring)

from atom-language-julia.

@garborg any update on this? I see that very little has happened over at atom/first-mate, but are some of your improvements independent of that?

from atom-language-julia.

Oh and @one-more-minute, I currently have this in my styles.less

 atom-text-editor::shadow {
   .string.docstring.julia {
     color: @syntax-result-marker-color;
   }
 }

It makes it more or less the same color as comments, depending on how the syntax uses that variable. It would be better to somehow pop the string scope when we enter a docstring though

from atom-language-julia.

The issue in atom/first-mate has been switched from 'bug' to 'matches macromates', which I imagine means won't fix. So dead end there until Atom supports more powerful tokenizers -- there are links in atom/first-mate#52 to issues suggesting supporting external tokenizers and/or Sublime 3's grammar.

Unfortunately, my plan to highlight almost everything correctly falls apart with a grammar that ignores 'end' patterns, so much of what I did will be useless. I do have plenty of new tests that should pass, but it'll take a bit of work to try to salvage actual changes.

from atom-language-julia.

Darn, that's disappointing.

I was looking forward to sane/complete scoping for everything.

from atom-language-julia.

The markdown specification used by language-gfm is that an italic string is started by an italic delimiter (_ or *) followed by anything that is not a space.

The following code works well:

"""
@switch `isa(x, _)` begin
  Integer; "x is an integer!"
  FloatingPoint; "x is a float!"
  "x is something else!"
end
Where `_` is replaced by the value for testing in each case. The final
"""
macro switch(args...)

from atom-language-julia.

This is the same as atom/language-gfm#44, btw.

from atom-language-julia.

Fixed on master.

from atom-language-julia.