Syntax Error: Unexpected & about skim HOT 9 CLOSED

@bartosz-zawada "html :AttributeMerger" was added earlier and this was made for support this feature https://github.com/judofyr/temple/blob/master/lib/temple/html/attribute_merger.rb

For your case I think it would be better and cleaner to write this:

- class_name = 'ruby'
div class = "hello #{ class_name }" Hello Ruby!

So that case class is defined only in one place.

Btw, I think it would be better to create github organization("skim" for example) though because otherwise there will many forks and nobody will support this in actual state.

from skim.

I'm getting the same error using code like this:

button.btn class=('active' if @active)

I suppose it's not a new feature 😄

from skim.

@amenzhinsky why not just to use this code:

button class = "btn #{ 'active' if @active }"

from skim.

Using class name after dot and with class directive simultaneously is confusing. I think it would be better to use it either with dot or with class directive only.

from skim.

@gkopylov I do understand in some cases it's confusing, but in examples like I posted above it would be nice to have. Maybe you're right 👍

from skim.

It's unfortunate because

.staticClass class=@dynamicClassName

is valid slim. Why deviate?

from skim.

I agree with @alubbe

We should maintain compatibility with slim because the main reason people use this gem is that they want to use slim for JS templating.

from skim.

I agree with @bartosz-zawada that we shouldn't use "html :AttributeMerger". Could you make a pull request with it?

from skim.

Any progress on merging @pavelkomiagin's PR?

from skim.