om

Please ask the channel first, not @dnolen directly!
sophiago 2017-04-20T01:28:40.408642Z

To be clear, it's rendering fine despite the warnings. The css transform just isn't being applied. I'm mainly just trying to do a sanity check that I'm getting the Om part right before I ask CSS people if it could be an issue with styling form elements in particular.

anmonteiro 2017-04-20T01:35:59.466727Z

@sophiago looking at it briefly you’re passing a CLJS map instead of a JS object to slider

anmonteiro 2017-04-20T01:36:04.467340Z

missing a #js

sophiago 2017-04-20T01:36:35.471419Z

around line 34?

sophiago 2017-04-20T01:38:32.486180Z

i'm getting several warnings around there, so suspect that's the problem.

anmonteiro 2017-04-20T01:41:24.508135Z

yeah

anmonteiro 2017-04-20T01:41:35.509412Z

also the apply call looks fishy to me

anmonteiro 2017-04-20T01:42:59.519946Z

actually you shouldn’t need #js there

anmonteiro 2017-04-20T01:43:07.520933Z

since you’re passing it to an Om component

anmonteiro 2017-04-20T01:43:30.523877Z

you’re just not using the style prop you’re passing in slider ever though

anmonteiro 2017-04-20T01:43:38.524936Z

so it’s normal the CSS transform is not being applied

sophiago 2017-04-20T01:44:26.531267Z

should i be calling slider as a function literal inside apply? and also setting :transform as a prop?

anmonteiro 2017-04-20T01:48:02.559636Z

probably

anmonteiro 2017-04-20T01:51:25.585727Z

sorry I wasn’t clear on the apply stuff

sophiago 2017-04-20T01:51:41.587883Z

is it not necessary?

anmonteiro 2017-04-20T01:51:43.588138Z

I meant just:

(dom/div nil (slider ...))

sophiago 2017-04-20T01:52:04.590714Z

ah ok

sophiago 2017-04-20T01:52:30.594292Z

also, to be clear...i'm unsure whether a matrix transform will even work on this element...that's why i hate CSS 😛

sophiago 2017-04-20T01:55:55.621499Z

i'm getting a runtime error just using ui as if it's a function call on line 25. but i believe you told me a while ago that's essentially how to use the defui macro works under the hood?

anmonteiro 2017-04-20T02:03:18.679218Z

@sophiago you still need to create a factory for it

anmonteiro 2017-04-20T02:03:31.680907Z

and the factory is what you’re supposed to call

sophiago 2017-04-20T02:05:43.697678Z

ok, thanks. i was originally using a factory but didn't realize i needed to use one in order to modify props like this

sophiago 2017-04-20T02:12:45.749554Z

@anmonteiro does the edited snippet above look correct to you?

sophiago 2017-04-20T02:32:09.886597Z

trying to get some feedback from the css people on freenode about transforming form elements, but

sophiago 2017-04-20T02:35:12.907584Z

apparently css isn't the issue here: https://jsfiddle.net/entuland/pLet0yb1/

sophiago 2017-04-20T02:36:48.918831Z

so now back to what i'm doing wrong with Om 😞

2017-04-20T08:21:51.686253Z

@sophiago In om.next, isn't it just :key and not :react-key?

pedroteixeira 2017-04-20T18:50:37.349984Z

hello! I was trying to find examples of combining om.next and datascript.. I see the use of a DataScriptStore implementing some protocols here https://github.com/swannodette/om-datascript/blob/master/src/om_datascript/core.cljs (2 years old) which seems "more complete" but the current wiki shows a different strategy: https://github.com/omcljs/om/wiki/DataScript-Integration-Tutorial (and I heard somwhere that set-query! would not work with datascript as state). Anyone care to comment if there is a current ref. best practice? Anyone used either implementation in production? trying to gather knowledge about past experiences

pedroteixeira 2017-04-20T18:51:44.374712Z

I'm aware about some of the trade-offs.. to perhaps favour keeping Om.Next state simply as an atom of projected data.. Considering those trade-offs right now 🙂

matthavener 2017-04-20T19:42:35.423882Z

@peeja: its possible to have multiple children of a component where some share idents.. not typically but certainly possible 🙂

peeja 2017-04-20T19:43:38.444876Z

Yeah, I guess that's true.

gardnervickers 2017-04-20T19:44:34.463439Z

@matthavener Yes we use that approach all over our app

matthavener 2017-04-20T19:45:11.475812Z

and your keyfn is some combination of ident + other stuff?

gardnervickers 2017-04-20T19:45:57.491956Z

I believe Om.next set’s a default one, we haven’t had any problems just not setting it.