React: O atributo "rowspan" não aparece no DOM
<th rowspan="2"></th> não define rowspan no 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>
);
}
Resultado:
...
<th data-reactid=".r[5je1f].[0].[1].[1].[6].[0].[0].[1].[0].[0].[0].[1]"></th>
...
Este pode ser meu primeiro PR? :)
ericclemmons
Todos 9 comentários
Se você fizer rowSpan isso funcionará - usamos a versão camelcase para atributos de consistência com a interface DOM onde você faria el.rowSpan = 2; . Já abri o # 255 para avisar ao usar a caixa errada, pois muita gente fica confusa com isso.
sophiebits
em 14 ago. 2013
Oh, é isso. Obrigado :) Vou aguardar a resolução do # 255 para o longo prazo, mas ajustarei meu código no curto prazo ...
ericclemmons
em 14 ago. 2013
Desculpe @ericclemmons , pode ter sido :) Eu tentei fazer uma passagem para atributos ausentes pré-0.4, mas se você notar alguma coisa faltando em https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfig .js , sinta-se à vontade para abrir um problema ou RP
zpao
em 14 ago. 2013
Oh, parece que rowSpan (camelCased!) Está faltando . Vou enviar um PR ...
ericclemmons
em 14 ago. 2013
Ops! Eu vi colSpan e ingenuamente presumi que rowSpan estaria lá também ... :)
sophiebits
em 14 ago. 2013
@ericclemmons Ainda está interessado em consertar isso?
zpao
em 23 ago. 2013
Fechando para # 291 ...
ericclemmons
em 25 ago. 2013
Visto que isso confunde as pessoas, em vez de ser consistente com o DOM, por que não ser consistente com HTML e oferecer suporte a atributos em minúsculas?
Eu só tive que voltar aos documentos para este problema sobre o atributo "autocomplete".
Estou escrevendo algo que parece HTML, deve funcionar como HTML ... caso contrário, é uma inconsistência, não consistência.
inimino
em 4 mai. 2014
Concordo. Seria preferível (para mim) se os atributos fossem todos mapeados .toLowerCase() .
ericclemmons
em 5 mai. 2014
Questões relacionadas
zpao
·
3Comentários
framerate
·
3Comentários
h1orz
·
3Comentários
MoOx
·
3Comentários
Prinzhorn
·
3Comentários
Comentários muito úteis
Se você fizer
rowSpanisso funcionará - usamos a versão camelcase para atributos de consistência com a interface DOM onde você fariael.rowSpan = 2;. Já abri o # 255 para avisar ao usar a caixa errada, pois muita gente fica confusa com isso.