Skip to content

Commit

Permalink
Use initial-config from devtools, not default-config
Browse files Browse the repository at this point in the history
Users can configure cljs-devtools via compiler options. For instance:

{...
 :builds
 {:app
  {...
   :dev
   {:compiler-options
    {...
     :external-config
     {:devtools/config
      {:min-expandable-sequable-count-for-well-known-types 1
       :features-to-install [:formatters :hints]}}}}}}}

Now, when users configure cljs-devtools, our jsonml
views (aka. "pods") will pick up on that.

For:
#416
  • Loading branch information
kimo-k committed Apr 13, 2024
1 parent 6456bdb commit 9ed6ab2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/day8/re_frame_10x/components/cljs_devtools.cljs
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
[day8.re-frame-10x.tools.reader.edn :as reader.edn]
[day8.re-frame-10x.panels.settings.subs :as settings.subs]))

(def default-config @devtools.prefs/default-config)
(def initial-config @devtools.prefs/initial-config)

(defn base-config
[]
Expand Down Expand Up @@ -64,7 +64,7 @@
;; TODO: If we expose ambiance and/or syntax color scheme as settings will need to fix this, maybe by recalculating
;; at the time the setting is changed/loaded.
(def custom-config
(merge default-config (base-config) #_bright-ambiance-config))
(merge initial-config (base-config) #_bright-ambiance-config))

(defn header [value config & [{:keys [render-paths?]}]]
(with-cljs-devtools-prefs
Expand Down

0 comments on commit 9ed6ab2

Please sign in to comment.