React: L'attribut "rowspan" n'apparaît pas dans le DOM

Créé le 14 août 2013  ·  9Commentaires  ·  Source: facebook/react

<th rowspan="2"></th> ne définit pas rowspan dans le DOM.

Composant:

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>
  );
}

Production:

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

Est-ce que c'est mon premier PR? :)

starter Bug
Source
picture ericclemmons

Commentaire le plus utile

Si vous faites rowSpan cela fonctionnera - nous utilisons la version camelcase pour les attributs de cohérence avec l'interface DOM où vous feriez el.rowSpan = 2; . J'ai déjà ouvert # 255 pour avertir lorsque vous utilisez le mauvais cas, car beaucoup de gens sont confus par cela.

picture sophiebits le 14 août 2013
❤5

Tous les 9 commentaires

Si vous faites rowSpan cela fonctionnera - nous utilisons la version camelcase pour les attributs de cohérence avec l'interface DOM où vous feriez el.rowSpan = 2; . J'ai déjà ouvert # 255 pour avertir lorsque vous utilisez le mauvais cas, car beaucoup de gens sont confus par cela.

sophiebits picture sophiebits le 14 août 2013
❤5

Oh, c'est ça. Merci :) J'attendrai la résolution de # 255 pour le long terme, mais ajuster mon code à court terme ...

ericclemmons picture ericclemmons le 14 août 2013

Désolé @ericclemmons , cela aurait pu être :) J'ai essayé de faire une passe pour les attributs manquants pré-0.4 mais si vous remarquez quelque chose qui manque sur https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfig .js , n'hésitez pas à ouvrir un numéro ou un PR

zpao picture zpao le 14 août 2013

Oh, il semble que rowSpan (camelCased!) Soit manquant . Je vais soumettre un PR ...

ericclemmons picture ericclemmons le 14 août 2013

Oops! J'ai vu colSpan et j'ai naïvement supposé que rowSpan serait là aussi… :)

sophiebits picture sophiebits le 14 août 2013

@ericclemmons Toujours intéressé à résoudre ce

zpao picture zpao le 23 août 2013

Clôture pour # 291 ...

ericclemmons picture ericclemmons le 25 août 2013

Étant donné que cela fait trébucher les gens, au lieu d'être cohérent avec le DOM, pourquoi ne pas être cohérent avec HTML et prendre en charge les attributs entièrement en minuscules?

Je devais simplement revenir à la documentation pour ce problème concernant l'attribut "autocomplete".

J'écris quelque chose qui ressemble à du HTML, cela devrait fonctionner comme du HTML ... sinon c'est une incohérence, pas une cohérence.

inimino picture inimino le 4 mai 2014
👍1

Je suis d'accord. Il serait préférable (pour moi) que les attributs soient tous mappés .toLowerCase() .

ericclemmons picture ericclemmons le 5 mai 2014
Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

bloodyowl picture bloodyowl  ·  3Commentaires
trusktr picture trusktr  ·  3Commentaires
kocokolo picture kocokolo  ·  3Commentaires
UnbearableBear picture UnbearableBear  ·  3Commentaires
jvorcak picture jvorcak  ·  3Commentaires