Anyone know of good reads when it comes to naming stuff? Specifically defining function args?

e.g. def point [x y] vs def point [[x y]]

readability vs conciseness vs expressivity

@bones https://leanpub.com/elementsofclojure

for the life of me i can't figure out how to construct a basic dropdown using this cljs ant-design react wrapper: https://github.com/priornix/antizer

(defn menu []
  (fn []
    [ant/menu
     [ant/menu-item "one"]
     [ant/menu-item "two"]
     [ant/menu-item "three"]]))

(defn dd [] [ant/dropdown [menu]]) ; No dice
(defn dd [] [ant/dropdown {:overlay [menu]}]) ; Nuh uh
(defn dd [] [ant/dropdown {:overlay menu}]) ; I didn't think this would work regardless, but nope!

I know I'm just being dumb here, but why is this wrong...

(->> parsed (:content) (-> (nth 3)))

the thread-first isn't pushing the result of (:content) as the first param to nth

the fact that nth order of args is different from (first) or (second)

@foamdino ^^ this is what that expands to 🙂

Fortunately, you can just use (->) all the way down

^^ swap (->> for (->

@foamdino macroexpand helps a lot with ->>

I'll have to remember that

I've had (-> to nested (->> before and it worked fine

Maybe the other way round?

(-> 10
    (->> (+ 4)))

The key thing is that ->> puts the argument at the end, so you were expanding with the result at the end (-> (nth 3) +it-went-here+)

What you wanted was (-> +it-went-here+ (nth 3))

yeah I expected that within the context of the thread-first it would put the argument in the correct place

@joshkh from the examples all of the components seem to be functions i.e.

(ant/menu 
        (ant/menu-item "one") 
        (ant/menu-item "two"))

see https://github.com/priornix/antizer/blob/master/examples/src/antizer_examples/rum.cljs#L278

whoops you are using reagent !!

ah yes, forgot to mention that part 🙂

@joshkh maybe lose the lambda in menu

it looks like the only difference between that and my third example is that the menu component is form1

fix menu

okay let me give it a shot

nope, same error: antd.inc.js:32809 Uncaught TypeError: Cannot read property 'props' of undefined

@joshkh This works

(defn menu []
   [ant/menu {:mode "vertical" :theme "dark" :style {:height "100%"}}
    [ant/menu-item "one"]
    [ant/menu-item "two"]
    [ant/menu-item "three"]])
 
 (defn dd [] [ant/dropdown {:overlay (r/as-element (menu))} [:a {:className "ant-dropdown-link" :href "#"} [:span "Choose me!"]]])

(r/as-element (menu)) being the core change

and possibly also setting a :mode on the menu - that is from debugging memory

thanks, @gmercer ! one small change i had to make - (reagent/as-element [menu]) rather than (menu) (... might be a rum thing). the dropdowns don't close though after being selected. weird. but that solves the reagent problem, thanks!

@joshkh I think you're mixing reagent and ŝablono here. That might be why you're having issues.

Unless you're explicitly using them together. Then you can ignore me 🙂

Hey what is the lighttable plug in for paran shortcuts?

Thanks @val_waeselynck