Node attr: colSpan is now colspan

[jadbox]

With Yolk v1 beta, I've had to change colSpan to colspan as otherwise the attribute gest removed from the DOM rendering.

[garbles]

This change might fix it? LMK

https://github.com/garbles/yolk/blob/master/src/propertyDescriptors.js#L63

-   colSpan: USE_EQUAL_SETTER | HAS_LOWER_CASE,
+   colSpan: USE_SET_ATTRIBUTE | HAS_LOWER_CASE,

[jadbox]

Okay, but I'm blocked for now until #89 as I don't have time to fork and rebuild the browser dist we're using atm.

[jadbox]

I've built against the latest version in master. I still have to use the lowercase colspan to set the property. 😕

https://rawgit.com/jadbox/yolk/better-distributable/dist/yolk.min.js

[jadbox]

itemProp also no longer seems to function, but in this case, itemprop also doesn't.

[jadbox]

Setting the data attribute to an object doesn't seem to property set the DOM attributes as it used to with the older version of Yolk. I just get data="[object Object]" as the result in the DOM instead of data-bar="foo"

Example jsx:

<tr data={{"bar":"foo"}} ></tr>

Out:

<tr data="[object Object]" ></tr>

Expected:

<tr data-bar="foo" ></tr>

update:
I see I can just do this now in jsx.. this works for me.

<tr data-bar="foo" ></tr>

[garbles]

Yea you have to use dashes. I didn't like that I was jumping through hoops to make the object case work, unfortunately.