just got bitten by the implicit refer all in the page macro again. while importing some code from another project, i failed to account for the free variable p below.

(elem :w (r 1 1) :h 66 :p p :c white)

ui.cljs:105Uncaught Error: Error validating attribute of type length with value function (var_args){
var rest__9347_SHARP_ = null;
if (arguments.length > 0) {
var G__9351__i = 0, G__9351__a = new Array(arguments.length -  0);
while (G__9351__i < G__9351__a.length) {G__9351__a[G__9351__i] = arguments[G__9351__i + 0]; ++G__9351__i;}
  rest__9347_SHARP_ = new cljs.core.IndexedSeq(G__9351__a,0);
} 
return G__9350__delegate.call(this,rest__9347_SHARP_);}.

@jumblerg: to avoid that I do most of work on .cljs files, only using .hl for pages that only calls other namespaces doing as little as possible.

yep, same

I even like to use h/p, instead of refering things.

but we shouldn’t have implicit magical things going on in the first place

kinda like going with use instead of refer

Easier to start, bites you later, it's a tradeoff.

yeah, classic easy vs simple scenario

nah, it's like 5 minutes work to make a task that implements your own page thing, transforms it into a hoplon page decl

as a workaround

But I sure took advantage of that when starting.

@micha: agreed, i’m just advocating an change to the page abstraction.

i’m not exactly sure what that change should be yet, i haven’t dug into the implementation, just making the general case for it.

(:refer-hoplon :exclude :all) could be added

is that supported now?

i don't know actually

lol

maybe?

heh. i’ll find out.

nope. made the compiler angry.

i think maybe it should go the other way, though… page should take a :`refer :all` sort of thing.

if you want the current behavior

:refer [div p video ...]

i think the solution is to make a new page macro

so we don't break existing things

agreed

this can be implemented first as a boot task

or as a macro that is shimmed into clojure.core

i'll make some experiments

k, yeah, i’ll look into it at same point.

or you could lol

happy fourth of july weekend

oh hey yeah

likewise!

(artifact "index.html"
  (:require
    [hoplon.core :refer :all]))

the ns macro wrapped with some extra sugar, basically

there’s also a bug i recall ignoring some time ago that pops up when there are multiple pages in the same project

what was the bug?

i’m trying to recall specifically, but it had something to do with one page loading in, then the wrong page loading in place of the original, maybe when reload was called, it could actually be an issue with that defaulting to index.html or something, not sure.

not the best of bug reports

but i know it was lurking out there somewhere, keep an eye out for it

lol ok

you can likely reproduce it by putting two different .hl files and their corresponding page macros with different *.html compilation targets in the same project.