Defmethod compile-element ::literal-tag-and-hinted-attributes

[ajchemist]

I think sablono can avoid some redundant type checking code with simple metadata hint, at runtime.

Here is some examples of rum component that its first arg attrs is always expected to be attribute data in cljs context.

(macroexpand-1
 '(rum/defc button < rum/static
    [& [attrs content]]
    [:button.mdl-button.mdl-js-button attrs content])) ; no ^:attrs hint
(def button "" (clojure.core/let [render-mixin__4606__auto__ (rum.core/render->mixin (clojure.core/fn ([& [attrs content]] (do (clojure.core/let [attrs50227 attrs] (clojure.core/apply js/React.createElement "button" (if (clojure.core/map? attrs50227) (sablono.interpreter/attributes (sablono.normalize/merge-with-class {:class ["mdl-button" "mdl-js-button"]} attrs50227)) #js {:className "mdl-button mdl-js-button"}) (if (clojure.core/map? attrs50227) [(sablono.interpreter/interpret content)] [(sablono.interpreter/interpret attrs50227) (sablono.interpreter/interpret content)]))))))) class__4607__auto__ (rum.core/build-class (clojure.core/concat [render-mixin__4606__auto__] [rum/static]) "button") ctor__4608__auto__ (clojure.core/fn [& args__4609__auto__] (clojure.core/let [state__4610__auto__ (rum.core/args->state args__4609__auto__)] (rum.core/element class__4607__auto__ state__4610__auto__ nil)))] (clojure.core/with-meta ctor__4608__auto__ {:rum/class class__4607__auto__})))

(macroexpand-1
 '(rum/defc button < rum/static
    [& [attrs content]]
    [:button.mdl-button.mdl-js-button ^:attrs attrs content])) ; ^:attrs hint
(def button "" (clojure.core/let [render-mixin__4606__auto__ (rum.core/render->mixin (clojure.core/fn ([& [attrs content]] (do (clojure.core/let [attrs50223 attrs] (clojure.core/apply js/React.createElement "button" (sablono.interpreter/attributes (sablono.normalize/merge-with-class {:class ["mdl-button" "mdl-js-button"]} attrs50223)) [(sablono.interpreter/interpret content)])))))) class__4607__auto__ (rum.core/build-class (clojure.core/concat [render-mixin__4606__auto__] [rum/static]) "button") ctor__4608__auto__ (clojure.core/fn [& args__4609__auto__] (clojure.core/let [state__4610__auto__ (rum.core/args->state args__4609__auto__)] (rum.core/element class__4607__auto__ state__4610__auto__ nil)))] (clojure.core/with-meta ctor__4608__auto__ {:rum/class class__4607__auto__})))

[r0man]

@ajchemist Could you please add some tests, then I would merge this?

[ajchemist]

@r0man sure. I've added some tests.

[r0man]

@ajchemist Released as 0.7.1. Thanks!