React: Das Attribut "rowspan" wird im DOM nicht angezeigt

Erstellt am 14. Aug. 2013 · 9Kommentare · Quelle: facebook/react

<th rowspan="2"></th> setzt nicht rowspan im DOM.

Komponente:

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

Ausgabe:

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

Kann das meine erste PR sein? :) :)

starter Bug

Quelle

ericclemmons

Hilfreichster Kommentar

Wenn Sie rowSpan tun, funktioniert es - wir verwenden die Camelcase-Version für Attribute, um die Konsistenz mit der DOM-Schnittstelle zu gewährleisten, in der Sie el.rowSpan = 2; tun würden. Ich habe bereits # 255 geöffnet, um zu warnen, wenn der falsche Fall verwendet wird, da viele Leute dadurch verwirrt werden.

sophiebits am 14. Aug. 2013

❤5

Alle 9 Kommentare

sophiebits am 14. Aug. 2013

❤5

Oh, das ist es. Danke :) Ich werde langfristig auf die Auflösung von # 255 warten, aber meinen Code kurzfristig anpassen ...

ericclemmons am 14. Aug. 2013

Sorry @ericclemmons , es hätte sein können :) Ich habe versucht, einen Pass für fehlende Attribute vor 0.4 zu machen, aber wenn Sie bemerken, dass etwas unter https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfig fehlt zögern Sie nicht , eine Ausgabe oder PR zu öffnen

zpao am 14. Aug. 2013

Oh, es sieht ganz wie rowSpan (camelcased!) Wird fehlen . Ich werde eine PR einreichen ...

ericclemmons am 14. Aug. 2013

Hoppla! Ich habe colSpan gesehen und naiv angenommen, dass rowSpan auch da sein würde… :)

sophiebits am 14. Aug. 2013

@ericclemmons Immer noch daran interessiert, dies zu beheben?

zpao am 23. Aug. 2013

Schließen für # 291 ...

ericclemmons am 25. Aug. 2013

Angesichts der Tatsache, dass dies die Leute auslöst, anstatt mit dem DOM konsistent zu sein, warum nicht mit HTML konsistent sein und Attribute in Kleinbuchstaben unterstützen?

Ich musste nur zu den Dokumenten für dieses Problem über das Attribut "Autocomplete" zurückkehren.

Ich schreibe etwas, das wie HTML aussieht, es sollte wie HTML funktionieren ... ansonsten ist es eine Inkonsistenz, keine Konsistenz.

inimino am 4. Mai 2014

👍1

Genau. Es wäre (für mich) vorzuziehen, wenn alle Attribute .toLowerCase() wären.

ericclemmons am 5. Mai 2014

War diese Seite hilfreich?

0 / 5 - 0 Bewertungen

React: Das Attribut "rowspan" wird im DOM nicht angezeigt

Hilfreichster Kommentar

Alle 9 Kommentare

Verwandte Themen