search

deadfoxygrandpa / Elm.tmLanguage

Elm language syntax highlighting and tool integration for ST2/3
https://packagecontrol.io/packages/Elm%20Language%20Support
MIT License
122 stars 27 forks source link

Syntax highlighting for `exposing` #43

Closed rtfeldman closed 9 years ago

rtfeldman commented 9 years ago

My first PR thanks to lots of guidance from @dnalot...feedback welcome! :smiley:

Before:

screen shot 2015-08-31 at 3 10 27 am

After:

screen shot 2015-08-31 at 3 12 56 am
texastoland commented 9 years ago

I'll look at this today and include it in beta!

texastoland commented 9 years ago

PS thank you I'd love to help you plan future features 👐

rtfeldman commented 9 years ago

@dnalot I have another branch with the awesome syntax highlighting (big hit at the office today...really makes things more readable!) - but it's based on this branch. Should I make a PR for that anyway or wait for this to get merged?

texastoland commented 9 years ago

Rebase your other branch onto this one and I'll look at both together!

texastoland commented 9 years ago

Closing in favor of #44. In the future you should be able to rebase onto the same branch and not open a new PR. The PR will just track the current branch :+1:

texastoland commented 9 years ago

Fixes #21 🎉