Compose Multiple Styles about spade HOT 6 CLOSED

Yes! You can compose multiple styles. I just updated the readme to clarify that you can either provide a single name or a collection of names.

from spade.

Hi,
I have some trouble using composes with a collection.
Using something like :composes [(style1) (style2)] give an "Invalid argument to :composes key:"
I did try to use :composes (seq [(style1) (style2)]) but then the name of the style that use compose is split as a collection

I think the problem come from here :

(->> (if (seq? composed)
           (into composed style-name)
           [composed style-name])

Using :composes (seq [(style1) (style2)]) if should be:
(into composed [style-name])
Or using :composes [(style1) (style2)]
it should be (flatten [composed style-name])

Or maybe I simply miss something here

What I do for now is :composes (clojure.string/join " " [(style1) (style2)] and it works nicely
Thanks for spade!
Samuel

from spade.

@ouvanous That does look like a bug! (conj composed style-name) should also work. I'll try to find time to fix this, but PR is welcome in the meantime ;)

from spade.

It's possible that the macro checking :composes expects a list and is mad about the vector, so may also need to tweak something there 🤔

from spade.

@ouvanous I just pushed a fix; should be available in the 1.2.0-SNAPSHOT. Let me know if that resolves the issue for you!

from spade.

I went ahead and published a stable 1.1.1 with the fix

from spade.