<th rowspan="2"></th>
no establece rowspan
en el DOM.
Componente:
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>
);
}
Salida:
...
<th data-reactid=".r[5je1f].[0].[1].[1].[6].[0].[0].[1].[0].[0].[0].[1]"></th>
...
¿Puede ser este mi primer PR? :)
Si lo hace rowSpan
, funcionará; usamos la versión camelcase para los atributos para mantener la coherencia con la interfaz DOM donde haría el.rowSpan = 2;
. Ya abrí el # 255 para advertir cuando se usa el caso incorrecto, ya que muchas personas se confunden con esto.
Oh, eso es todo. Gracias :) Esperaré la resolución del # 255 a largo plazo, pero ajustaré mi código a corto plazo ...
Lo siento @ericclemmons , podría haber sido :) Intenté hacer un pase para los atributos faltantes anteriores a 0.4, pero si nota algo que falta en https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfig .js , no
Oh, parece totalmente que falta rowSpan
(camelCased!). Enviaré un PR ...
¡Ups! Vi colSpan e ingenuamente asumí que rowSpan estaría allí también ... :)
@ericclemmons ¿ Todavía
Cerrando por # 291 ...
Dado que esto desconcierta a la gente, en lugar de ser coherente con el DOM, ¿por qué no ser coherente con HTML y admitir atributos en minúsculas?
Solo tuve que volver a la documentación para este problema sobre el atributo "autocompletar".
Estoy escribiendo algo que se parece a HTML, debería funcionar como HTML ... de lo contrario, es una inconsistencia, no una consistencia.
Estoy de acuerdo. Sería preferible (para mí) si todos los atributos estuvieran asignados .toLowerCase()
.
Comentario más útil
Si lo hace
rowSpan
, funcionará; usamos la versión camelcase para los atributos para mantener la coherencia con la interfaz DOM donde haríael.rowSpan = 2;
. Ya abrí el # 255 para advertir cuando se usa el caso incorrecto, ya que muchas personas se confunden con esto.