cljsrn

https://github.com/drapanjanas/re-natal | https://github.com/drapanjanas/re-natal/wiki/FAQ | https://github.com/condense/mercury-app/wiki | https://github.com/seantempesta/expo-cljs-template/ https://www.npmjs.com/package/create-expo-cljs-app
Oliver George 2020-05-03T02:11:57.439700Z

Righto. My issue is with react-navigation. Here is a repro repo: https://github.com/olivergeorge/krell-repros/tree/react-navigation-3

Oliver George 2020-05-03T02:12:55.440700Z

If I change something in the view ns like the container1 text it's not displayed. If I cause the core namespace to reload (say add some whitespace and save the file) then the view updates.

Oliver George 2020-05-03T03:37:23.441400Z

For completeness I've done the same for v4 and v5. Same issue.

Oliver George 2020-05-03T04:02:34.442400Z

(slight variations on which file needs to change to get the view to update on v5)

Oliver George 2020-05-03T06:00:36.444Z

I think I have a solution, via re-render with key prop...

(def cnt (r/atom 0))

(defn app []
  [navigation-screen/container {:key @cnt}])

(defn ^:export -main [& args]
  (swap! cnt inc)
  (r/as-element [app]))
Still thinking this through. I suspect it will wipe component local state across the app - not sure if that's a problem or not but makes it a bit of a crude implement.

dnolen 2020-05-03T15:30:59.445400Z

solving interactions with various components doesn't seem practical for Krell - I simply haven't seen this particular problem in our app

dnolen 2020-05-03T15:33:16.446900Z

the thing I'm interested in is why this wasn't a problem for you in re-natal - or maybe you got far enough along where you stopped noticing?

Oliver George 2020-05-03T21:09:28.448800Z

I’m not sure about that either. Wondering if the caching Is more aggressive on recent version of react / reach native.

Oliver George 2020-05-03T21:09:43.449300Z

I’ll try for a re-natal repro

Oliver George 2020-05-03T21:31:59.451Z

Also wondering if figwheel is reloading dependent namespaces. E.g. changing views causes nav to reload.

dnolen 2020-05-03T23:02:46.452Z

currently we're not reloading dependent namespaces - though that's trivial to add now

dnolen 2020-05-03T23:03:56.452700Z

we also don't handle all the macros cases - though in my opinion this is not necessary for the alpha

dnolen 2020-05-03T23:04:13.453100Z

dependent namespaces probably