Evaluating cljs.core/macroexpand-1 for a / project in a buffer differs from the result in a buffer. E.g. given a file:

(ns om-tutorial.core
  (:require [om.dom :as dom]
            [om.next :as om :refer-macros [defui]]))

(defui HelloWorld
  Object
  (render [this]
          (dom/div nil (get (om/props this) :title))))

(do
 (do
  (clojure.core/defn
   HelloWorld
   []
   (cljs.core/this-as
    this__41180__auto__
    (.apply js/React.Component this__41180__auto__ (cljs.core/js-arguments))
...

Hi

I have an issue with clojurescript 1.9.946

when I try to compile my codebase with 1.9.908 I have no worries, but when I do with 1.9.946 I get the following error : No reader function for tag Inf

I guess it comes from Clojure 1.9.0 new tag litterals for Inf and NaN. But I don't see why there was no problem with 1.9.908

Looks like CLJS-2352 should had fixed that. Maybe I have an old lib somewhere

@chu this is cause by an outdated tools.reader version somewhere in your deps

you can either add [org.clojure/tools.reader "1.2.1"] to your deps directly to ensure you get a compatible version

or upgrade whichever library is bringing in the old one

lein deps :tree should complain about a conflict

I'm using Figwheel and nREPL, but I don't understand how importing namespaces works in clojurescript. I use (ns myns) to change to my ns, but any of my symbols I try to evaluate are reported to be undefined

@roklenarcic (in-ns 'myns) is what you want

(ns ...) is only for defining namespaces

thanks

Does anyone know the trick to getting gen/generate working in cljs? Here's what I'm trying:

cljs.user> (require '[cljs.spec.alpha :as s])
nil
cljs.user> (require '[cljs.spec.gen.alpha :as gen])
nil
cljs.user> (gen/generate (s/gen int?))
#object[Error Error: Var clojure.test.check.generators/simple-type-printable does not exist, clojure.test.check.generators never required]

[org.clojure/clojure "1.9.0"]
[org.clojure/clojurescript "1.9.946"]
[org.clojure/test.check "0.9.0"]

@thheller Thanks ! I'll try asap to add a recent tools.reader

It worked 👌

In theory - should I be able to add material-ui-next as a npm-deps compiler option and be able to use it? Not sure if it picks up the module type correctly

in my experience npm-deps is kind of a crap shoot. it apparently depends on how that module specifies its require statements. for instance, i tried to use it with react-dnd and it dies (I think) because that module has a require('react') statement in its UMD header. but at any rate I couldn’t get it to work so i just used foreign-libs

@lee.justin.m Not sure what you mean, if module works with CommonJS/ES6, it should work with npm-deps, even if it uses UMD module (which react-dnd doesn't seem to use)

(should work = if Closure-compiler and ClojureScript were complete, which they aren't currently)

really? the fully built minified version that you grab off of a CDN seems to have UMD header

But npm-deps won't use that file, it will use one from npm

Which is just CJS module compiled by Babel or something: https://unpkg.com/[email protected]/lib/index.js

i dunno. i’m not an expert in all of these competing module standards (and, ironically, I never had to be to get things done in javascript). all i know is that i added

:npm-deps {:react-dnd "2.4.0"}
:install-deps true

Error: Can't resolve 'react' in

Ah yes, the npm package also contains browserified UMD module under dist/ but package.json main property points to lib/index.js so that is what will be used

React-dnd lists React as peer-dependency, so it won't get automatically installed, you need to list React on :npm-deps also

I see. There’s a difference between having it in your normal dependencies and your npm-deps I guess? Basically this is what I tried before I gave up and tried it a different way: https://gist.github.com/jmlsf/d691e53e1fea4019a393412f781e2561#try-1-lets-use-npm-deps

it was super hard for me to figure out what npm-deps was actually doing

Normal dependencies would be the cljsjs/react foreign-lib? Foreign-libraries and Node packages won't work together

i mean, i’m using something like reagent, which is already importing react. i guess I should have excluded it and then imported it using npm-deps? but it isn’t clear to me if reagent would then be able to get to react if i import react using npm-deps

Reagent 0.8 will fix this, you can check the changelog for details

can i ask a really naive question about clojurescript? does it "merely" compile to js or are there also libraries for dom manipulation similar to jquery?

@michael740 clojurescript itself is just a compiler. But it interoperates with JavaScript so you can certainly manipulate the dom using jquery itself, or react. There are cljs libs like reagent and rum that wrap react to look like more idiomatic clojure code.

@lee.justin.m Do you know if there are any tutorials for reagant or rum that show Clojurescript along side Javascript that would accomplish the same goal?

What do you mean by alongside?

I mentioned something to this effect in the #cljsrn room, but Im having trouble seeing how the two equate. Im pretty new to clojurescript but have a decent amount of Scheme / Elisp experience, And im pretty fresh in the React world

Like… Here is a simple react App / Component that does x and here is the equivalent component in clojurescript / reagant

A great one would be tic tac toe game from the official react tutorial

I'd also be interested to see something like that

something even simpler might be helpful, too - simpler dom manipulation / event handling, for example

(please excuse these comments if this is not the proper channel or forum for this topic)

well, there’s the infamous todomvc example. you could compare the various pure react flavors: http://todomvc.com/examples/react/#/ with the various cljs flavors: https://github.com/gadfly361/cljs-todomvc

which is lighter-weight, om or reagent?

reagent

i do everything in reagent, and I think these super complicated frameworks are totally the wrong way to do things if you are just learning. they add about 6 layers of indirection to everything, which is only beneficial if you have a large app and/or a large team. fulcro is a bit different since it is trying to do the whole stack from database to frontend in an idiomatic clojurey way, but i haven’t used it

reagent looks good. it looks super plain. i guess you'd need to be familiar with react/vue conventions, but still

in the rendered js, is it easy to expose an interface?

@michael740 Look at https://purelyfunctional.tv/guide/reagent/ and http://reagent-project.github.io/

use figwheel template and start playing around. That's that fastest way to get a handle on reagent

@bfast you know it’s a bit unfortunate that guide is sitting out there on a third-party website. that guide is basically what i had in mind in terms of submitting to reagent. that’s basically what the reagent readme should look like

Is it possible to mock a function before you require it in your test?

I have a function that executes right when the namespace is loaded, and i’d like to mock it out

never run code at the top level of a namespace

(other than defining constants with no side effects, or functions you might call later of course)

you can use with-redefs, but that doesn't work before loading the namespace containing the definition

and even with-redefs has pitfalls

Thanks. The reason I’m doing it is b/c it’s a lambda function, and I’d like to run some setup code right when the function boots. I can probably stick it in the handler function though. What sort of pitfalls?

it is a global change - and come to think of it it probably doesn't even work in cljs

I’ve used with-redefs in clojurescript

it's less tricky in cljs than clj - the main drawback would be that laziness that escapes the block would call the original definition if realized later

but with-redefs can't mock out a function in code that hasn't even loaded yet

Ya, I noticed that while I was writing tests. Future tests would continue using the mocked version. I forgot what I had to do to fix it.

Thanks! I’ll see if lambda itself has an onload i could hook into

if some function needs other setup to run, that would be the job of your top level code, to set up any initialization needed. Putting it at the top level of the namespace breaks things like tests.

Thanks!

I'm having trouble creating a reagent component that can accept other components as parameters. Is that something that is supported?

a common idiom is to have a -init function for each namespace that needs it, with documentation that a consumer must call it before using applicable code

Yep. How are you passing them?

(defn simple-node [node-content & children]
  (let [node-state (r/atom {})]
    (fn [node-content & children]
      [:li.tree-node
       [:div.tree-content node-content]
       [:ul.tree-body
        (doall (for [i (range (count children))]
                 (let [child (nth children i)]
                   (with-meta child {:key i}))))]])))

I am trying to create a tree view where each node is a component with its own internal state (expanded, collapsed, etc.)

what’s with-meta look like?

with-meta?

Oh, it’s a built in function

I'm using that to attach keys to the children after the fact, since they don't know their index when provided

This is what a use-case looks like

[simple-node [:p (session/get [:a])]
 [simple-node [:p (session/get [:b])]
  [simple-node [:p (session/get [:c])]]]]

Ideally, I'd want each to update independently as their corresponding values change, so I don't have to rerender all children

Could you try wrapping the last line in []?

[(with-meta child {:key i})] or [with-meta child {:key i}]?

the first one

Uncaught Error: Invalid arity: 0

My current use case, as shown above, will render initially, but when I change a referenced value in the session, I get either:

Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.

or

Uncaught TypeError: Cannot read property 'getHostNode' of null

I can get something like this to render without errors https://codepaste.net/c5djck , but updating the value one node references, will cause all nodes to re-render