This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2018-01-18
Channels
- # architecture (25)
- # beginners (57)
- # boot (3)
- # cider (38)
- # clara (6)
- # cljsrn (6)
- # clojure (54)
- # clojure-china (4)
- # clojure-greece (1)
- # clojure-italy (3)
- # clojure-romania (1)
- # clojure-russia (7)
- # clojure-spec (68)
- # clojure-uk (46)
- # clojurescript (73)
- # community-development (2)
- # core-async (7)
- # cursive (17)
- # datomic (143)
- # duct (2)
- # emacs (12)
- # events (5)
- # figwheel (3)
- # fulcro (15)
- # hoplon (19)
- # jobs (12)
- # jobs-discuss (85)
- # nginx (3)
- # off-topic (111)
- # onyx (7)
- # other-languages (1)
- # re-frame (30)
- # reagent (19)
- # remote-jobs (1)
- # ring (7)
- # rum (1)
- # shadow-cljs (18)
- # spacemacs (4)
- # specter (4)
- # sql (24)
- # test-check (1)
- # unrepl (10)
- # vim (6)
- # yada (1)
@ramblurr in the second example you create an instance of a component and passing a hash of attributes, the component itself should read that attribute and pass it into an element
does that mean I should design every component such that it can expect the first argument to be a hash of attributes?
yes, at least in my experience it’s a good pattern
I’m having trouble getting a (previously working) reagent app running. In the browser I see:
Cannot read property 'render' of undefined
at reagent$dom$render_comp (dom.cljs?rel=1516289963109:20)
Using reagent 0.8.0-alpha2
If I do window.ReactDOM
in the browser, I see it
@colindresj Need for information. How are you providing React JS? Did you check the upgrade guide?
I did take a look @juhoteperi
Have been using version 8 for a while now without problems, making this a bit weird
Bringing in react and all its siblings via cljsjs
Cljsjs should work. Check your deps to validate you have at least the same version as what Reagent depends on, or later.
Versions look good to me
I actually got this to build correctly but had to require cljsjs.react.dom
myself, which I wasn’t doing before
@colindresj Strange. That shouldn't be needed.
Yeah that’s what I thought too, but that’s the only way I got it to work