Heex support

[joeesteves]

Add support for heex (single curly braces) <div id={@id}> , skip interpolation strings inside Heex expressions.
Address 👇 #555

Before:

After:

[joeesteves]

Hi @jbodah, as u may notice I'm not a vim-script expert. Don't know if this is the correct approach for solving this, just wan't to share my workaround.

I'll be glad to help improving heex support. So any guidance from the community will be welcome

[sodapopcan]

Thanks for taking this on @ponyesteves—this has been driving me a little nuts!

The only comment I have on your PR is that you forgot to uncomment the guard clause (I'm not the owner of the repo, of course, I just follow it).

Also, just looking at your screenshot, did you know you can pass a list to the class attribute to avoid string interpolation? Maybe you do know and prefer the interpolation but couldn't keep myself from commenting just in case you didn't as I quite the list version :)

<div class={["list-item_group", "text-secondary", maybe_hidden_balance_class(balance)]}>

It's also nice because if maybe_hidden_balance_class returns nil and it's the only entry, class="" doesn't even get rendered (if you care about that kind of thing).

[joeesteves]

The only comment I have on your PR is that you forgot to uncomment the guard clause (I'm not the owner of the repo, of course, I just follow it).

Hey @sodapopcan thanks for your suggestion, I've address it. It seems I missed on removing my trial and error comment. I was trying to find out how all this vim scripting works, so great catch.

TIL 👇

<div class={["list-item_group", "text-secondary", maybe_hidden_balance_class(balance)]}>

Great advice on using Lists to avoid String interpolation. I'm migrating some of my projects to phoenix 1.6 and I wasn't a Surface user, So this heex syntax is kind of new for me.

[Matsa59]

Any news ?

If you want to continue using this repo I add the following script to my init.vim (I'm on neovim) as a workaround

au BufEnter *.heex syn region  surfaceExpression matchgroup=surfaceDelimiter start="{" end="}" contains=@elixirTop  containedin=ALLBUT,@eelixirRegions keepend
au BufEnter *.heex syn region  surfaceExpression matchgroup=surfaceDelimiter start="{" end="}" skip="#{[^}]*}" contains=@elixirTop  containedin=htmlValue keepend

I just copy the changes from the PR
I don't know anything from vim script so I hope I did nothing wrong x)

[jbodah]

Hi all - thanks for the patches. I can merge however I do not intend to much more support for vim-elixir going forward as I have been far removed from Elixir for several years now. If anyone would like to take the maintenance reins then please let me know and I can coordinate with the Elixir team to have you added as a maintainer

[jbodah]

Was not able to get new unit tests for heex working on this, however tested manually and confirmed no regressions in existing suite

[joeesteves]

Hi all - thanks for the patches. I can merge however I do not intend to much more support for vim-elixir going forward as I have been far removed from Elixir for several years now. If anyone would like to take the maintenance reins then please let me know and I can coordinate with the Elixir team to have you added as a maintainer

@jbodah Thanks for merging this.