React: El atributo "rowspan" no aparece en DOM

Creado en 14 ago. 2013  ·  9Comentarios  ·  Fuente: facebook/react

<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? :)

starter Bug

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ía el.rowSpan = 2; . Ya abrí el # 255 para advertir cuando se usa el caso incorrecto, ya que muchas personas se confunden con esto.

Todos 9 comentarios

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() .

¿Fue útil esta página
0 / 5 - 0 calificaciones