@anmonteiro any chance that you could cut a new alpha release of compassus including this change? https://github.com/compassus/compassus/commit/2122406a26392d840dd9df3daf8645de520677e6

is using parser inside the env argument in the parser functions intended for running queries within the parser functions?

levitanong: I think so.. for recursive parsing

For server side rendering, the content inside a (dom/script) is being html-escaped. Is this correct? Is there a way to avoid this? Example

(dom/render-to-str (dom/script nil "foo('bar');"))
=> "<script data-reactroot=\"\" data-reactid=\"1\" data-react-checksum=\"-1720051763\"> foo(&#x27;bar&#x27;);</script>"

@danburton done! https://twitter.com/anmonteiro90/status/796797676633341952

@petterik probably a bug. We need to check how React acts in that case

@anmonteiro thanks

@petterik hrm:

(dom/render-to-str (dom/script nil "'foo bar'"))
"<script data-reactroot=\"\" data-reactid=\"1\" data-react-checksum=\"1770461025\">&#x27;foo bar&#x27;</script>"

seems like the correct behavior

(this is at the node repl using react dom server)

hmm

chrome complains about Unexpected token &

you'll have to do it like this:

cljs.user=> (dom/render-to-str (dom/script #js {:dangerouslySetInnerHTML #js {:__html "'foo bar'"}}))
"<script data-reactroot=\"\" data-reactid=\"1\" data-react-checksum=\"-1734339355\">'foo bar'</script>"

(When it gets the html from the server, generated with dom/render-to-str)

let me try

https://facebook.github.io/react/docs/dom-elements.html#dangerouslysetinnerhtml

cool! that worked. Thanks!

👍

Actually no. My errors went away, because it renders it as nothing 🙂

(dom/render-to-str (dom/script {:type "text/javascript" :dangerouslySetInnerHTML {:__html "'foo'"}}))
=> "<script type=\"text/javascript\" data-reactroot=\"\" data-reactid=\"1\" data-react-checksum=\"297998988\"></script>"

@petterik oh.

we might not handle dangerouslySetInnerHTML in Om

open an issue please 🙂

@anmonteiro will do!

thanks!

Created https://github.com/omcljs/om/issues/818

@petterik https://github.com/omcljs/om/pull/819

@anmonteiro Verified that it works with a simple string and in our app with more complex examples! Thanks creating a patch so fast!

⚡

awesome 👍

question, should I close the issue now that I've verified it? Or is that something you do when the pull request has been merged?

should be closed when the PR is merged

it's OK to leave it open

hi. is it possible to wrap a defui in a function in order to pass it params? i think i may be getting error because of this: "Map literal must contain an even number of forms", either that or i need to bind them somehow?

@sophiago you can do that but ui exists to make anonymous components. i.e.:

(ui
  Object
  (render [this]
    ...))

the error you're getting has nothing to do with any of this, you're most likely creating a map with an odd number of forms 🙂

which means you don't have a value for some key

ah, right i could have used ui

ah, interrupted one sec

sorry...anyway, the reason i asked that specific question was because i figured maybe the param wasn't being passed through hence resulting in the mismatch

since it's the argument to a selector used as the value to one key

@sophiago hrm something wrong with the syntax there?

matrix(args) is JS, (matrix args) is ClojureScript

ah ok

i've never used css matrix transforms in js either tbh

usually use canvas

well, it's not about matrices

it's about Lisp syntax

well seems obvious now

you probably also want to drop the # I think

unless you want the div child to be a function

but I suspect you want the result of the function

ah yes, i had that from calling it as the function in a map

but not what i want now

👍

huh, figwheel has not been compiling my code live

not in cases like this at least

if you have an error Figwheel won't compile

when the initial pass fails

it should show you its HUD with the error

it gives you a screen that is misleading to that effect

but the command line would seem to say run it again, which is what i do

really exciting now to see how om.next interops with this decent sized library i wrote

hmm, it's compiling. but i switched to ui and not i'm getting Use of undeclared Var core/ui and errors from the following lines that are also replicated in the js

that doesn't make any sense to me

oh, easy

i didn't include it

not using the correct terminology there. in the require brackets?

@anmonteiro the matrix syntax...that's not core.matrix you were thinking, but js interop, right? because that seems to be the last warning i'm getting

it might be the case i just need to quote it and have the symbol "matrix" at the front of the list?

@sophiago uh as I said I didn't mean "matrix syntax", but lisp syntax in general

I'm not sure what you're trying to achieve, but if matrix is a function it needs to be inside the parens

(matrix ...)

it does seem like that from the css, but doesn't really make sense to have a function inside a selector?

oh.. is matrix a CSS thing?

then everything should probably be inside a string

"matrix( ... )"

oh no you're correct

it's a function

but you're saying it's a function passed as a literal?

a string literal i mean, not a clojure function literal

https://developer.mozilla.org/en-US/docs/Web/CSS/transform-function

hrm, so if it's CSS you need to pass it as a string

like you'd do in plain JavaScript

ok, thanks. i would never have figured that out

np

to pass the function as a string i mean

ok, seems to have done it

still not behaving correctly, but error is gone

the next thing is putting #js in front of the map

and now i have to run to go see rich talk about spec since it'll likely be standing room only and i tend to be late to those things

oh, missed that

oh it's not

sorry

yeah, there's no map

right?

you only need to pass #js to dom/ stuff

yeah don't need it there