@pupeno question for you about FreeForm. When you toggle a checkbox the event is updated with the value, i.e. you can't distinguish between checking & unchecking. In general of course this shouldn't be a problem since there should be no way for the form and your "in db" model to get out of sync although in testing i've managed to do this a couple of times (possibly related to Chrome's reloading issues)

@pupeno the question is, is it intentional? Or would it be better for a checkbox to send .-checked instead of .-value?

Interesting. Let me see.

(or gate sending .-value/`nil` based on .-checked perhaps)

Ah... right... I haven't even worked on checkboxes yet.

at the moment i am just using the event as a trigger for update not which is fine but makes me uncomfortable

I think the checkbox should generate it's current state and not the toggle.

as i say, i have managed in testing to get them out of sync

That's in-sync with the rest of the functionality anyway.

So, no, it's not intentional and I'd tend to agree with you (without looking too much into it).

probably a relatively simple patch to bind-input i could have a crack at it if you like, but happy to leave to you if you'd prefer

I'm happy for you to work on it, give me a couple of hours to clean up free-from (it was in the middle of a refactor that should be abandoned).

I will work a lot on Free form once I release my startup product, so, in the next couple of months, but not sure when.

okay, i'll get something working here first

@pupeno you might want to remove your .idea folder from the free-form repo

also the .iml file

Is it bothering you in some way? The reason it's there is so that Cursive users will get code style, testing profiles, etc without having to do any work.

oh if it's deliberate that's fine (I use Cursive)

It is deliberate but not battle tested. If it's annoying other people, not working, etc, then I'll remove it.

Hrmm... Error synchronising com.pupeno/free-form:0.6.0-SNAPSHOT: Could not find artifact tech.dashman:reagent-toolbox:jar:0.1.0-SNAPSHOT in clojars (https://clojars.org/repo/)

Yes... that's the issue.

I'm starving, but give me a sec to try to fix it.

Go, eat 🙂

This can wait

Try again.

I mean, do a git pull.

@pupeno looks good.. thx

Ok.

oh you've already made the edit 🙂

I think someone else did it, but it wasn't released.

We should add the checkbox and radio buttons to the examples.

I'll grab some food and look into this. It might be time for a release.

brb

@pupeno checkbox functionality working here

it's a version jump so i'm not sure what else may have changed

but that bit seems good

I'm back.

@sandbags I try to keep the changelog complete.

oh i didn't see one

It's at the bottom of the readme.

🙂

sry

Ah... we do have checkbox and radio buttons in the example, I just never pushed it.

I'm going to do a 0.6.0 release with that.

There we go: https://clojars.org/com.pupeno/free-form/versions/0.6.0

@sandbags checkboxes and radio buttons: http://free-form-examples.pupeno.com/reagent/bootstrap-3

isn't it annoying that subscribe always expects a vector? why can't it wrap it in one automatically when there's a single key?