Hi. In the ratom.cljs file there is a make-reaction function which returns an instance of Reaction type (->Reaction f nil true false nil nil nil nil), where f is reagent render function. I can't see/understand how an referenced atom inside that f render function causes the f to execute (causing the rerender). In general add-watch function sets some function to execute when some atom changes. Based on this, i can only think that when creating a reaction the f function should be parsed as list data structure, all atoms in it should be found and watches then set on those atoms indicating to execute that f function when they change.

Anyone seen this: reagent.dom.node$module$react_dom is undefined?

Just started a brand new Reagent project, lein new figwheel => add reagent and bump clojurescript to latest, and now I get this

seems it's trying to use reagent/reagent-dom from npm-deps, whereas I'd be happy if it just used cljsjs

managed to get it working by adding the latest react/react-dom packages. reagent 0.8.0 depends on 16.3.0-1, whereas it worked with 16.3.2-0

@tomaas Reagent catches all RAtom deref calls during render and adds watches from components to the ratom

or watch from other ratom to another

the deref method uses dynamic var to keep track of the context

e.g.

(defn foo-component []
    (if @a-ratom
       [:h1 @b-ratom]
       [:div @c-ratom]))

Reagent will only watch a-ratom, and then b or c depending on a state

https://github.com/reagent-project/reagent/blob/master/src/reagent/ratom.cljs#L51

got it! thanks a lot @juhoteperi!

Getting the following error when trying to upgrade from reagent .7 to .8

No such namespace: react, could not locate react.cljs, react.cljc, or JavaScript source providing "react"

Seems like it just cannot find react

What environment are you targetting?

https://github.com/reagent-project/reagent/blob/master/docs/0.8-upgrade.md

browser

Hmm may need to provide React directly via cljsjs? I'm not sure why though

actually, may be a cache issue? https://clojurians.slack.com/archives/C0620C0C8/p1525208684000081

Yeah, looking into the cache

got it to build...all the cljsjs/* deps needed to be upgraded

As mentioned in the guide: > Using Reagent with Cljsjs packages doesn't require changes, other than making sure you update Cljsjs React dependencies, if you have direct dependencies to them. Also, using other packages depending on Cljsjs/React could cause one to have old version.

huh .. i just noticed that an element [foo {:style {:background-image ...}}] causes react to complain did you mean "backgroundImage", but it still works? does reagent not kebab->camelCase :style maps?