"rowspan" attribute does not appear in DOM

<th rowspan="2"></th> does not set rowspan in the DOM.

Component:

render: function() {
  return (
    <thead>
      <tr>
        {this.props.columns.map(this.renderColumnLabel)}
        <th rowspan="2"></th>
      </tr>
      <tr>
        {this.props.columns.map(this.renderColumnControl)}
      </tr>
    </thead>
  );
}

Output:

...
<th data-reactid=".r[5je1f].[0].[1].[1].[6].[0].[0].[1].[0].[0].[0].[1]"></th>
...

Can this be my first PR? 😃

[sophiebits]

If you do rowSpan it’ll work – we use the camelcase version for attributes for consistency with the DOM interface where you’d do el.rowSpan = 2;. I’ve already opened #255 to warn when using the wrong case since lots of people get confused by this.

[inimino]

Given that this trips people up, instead of being consistent with the DOM, why not be consistent with HTML and support all-lowercase attributes?

I just had to go back to the docs for this issue about the “autocomplete” attribute.

I’m writing something that looks like HTML, it should work like HTML… otherwise it’s an inconsistency, not consistency.