This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2017-12-23
Channels
- # adventofcode (92)
- # beginners (12)
- # boot (3)
- # boot-dev (38)
- # cider (9)
- # clara (26)
- # cljs-dev (26)
- # cljsrn (3)
- # clojars (3)
- # clojure (48)
- # clojure-france (6)
- # clojure-greece (2)
- # clojure-spec (3)
- # clojurescript (7)
- # datomic (3)
- # fulcro (6)
- # hoplon (19)
- # jobs-discuss (1)
- # leiningen (4)
- # lumo (9)
- # off-topic (8)
- # onyx (43)
- # planck (10)
- # powderkeg (4)
- # precept (44)
- # re-frame (4)
- # shadow-cljs (4)
- # sql (13)
- # uncomplicate (1)
- # unrepl (3)
hmmm, can we also move the experimental stuff into a new ns in 7.2+?
yes, thats better than what I had in mind which was to pull it entirely
we need a change log also ๐
so much to do
we could pull it and move to a different repo
i haven't personally used the experimental stuff so i don't really know much about it
it would fit inline more within hoplon/brew
really hoplon.core should be all the stable ancient code ๐
yeah, less is more i think
so i think the simplified but more consistent logic for these fns should just be "use managed version if parent is already managed or any child is a cell, else use native prototype fn"
in my branch all the overrides now look something like this
(defn- set-appendChild!
[this kidfn]
(set! (.-appendChild this)
(fn [child]
(this-as this
(if (or (managed? this) (cell? child))
(managed-append-child this child kidfn)
(.call appendChild this child))))))
@flyboarder i'm finding lots of edge cases here >.< that refactor is looking more and more attractive
i suspect that in the refactor that insertBefore
and replaceChild
overrides can just be immediately deleted - they don't seem to work for cells and aren't currently used for anything internally AFAICS
Ok everyone, im going to press the big buttonโฆโฆ. any last objections!?
๐ It has been done [hoplon/hoplon "7.1.0"]
is on Clojars! ๐
A ๐ from @thedavidmeister and myself ๐
this is great news, thank you and congratulations!