halohalospecial / atom-elmjutsu

A bag of tricks for developing with Elm. (Atom package)
https://atom.io/packages/elmjutsu
MIT License
192 stars 24 forks source link

Support non-ASCII names #57

Closed jespersm closed 7 years ago

jespersm commented 7 years ago

Elmjutsu doesn't seem to work with "international" letters, i.e. outside the ASCII ranges, even if Elm, Atom and JavaScript support these just fine.

Example:

med = " med "

fløde = "fløde"

dessert = "rødgrød" ++ med ++ fløde

Hyperclicking on "med" jumps to the definition just fine, but doing the same on "fløde" is a no-op. These should work the same.

jespersm commented 7 years ago

Awesome, thanks! I did actually download the code and intend to fix it myself, but wasn't sure I had the rights commands to build the elm/.js files from their .elm counterparts. Is there an easy way?

halohalospecial commented 7 years ago

I just execute elm-make Indexer.elm --output=indexer.js from the command line :grinning: However, the change I did was in a .js file.