React: 「rowspan」属性はDOMに表示されません

作成日 2013年08月14日  ·  9コメント  ·  ソース: facebook/react

<th rowspan="2"></th>は、DOMにrowspanを設定しません。

成分:

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

出力:

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

これは私の最初のPRになることができますか? :)

starter Bug

最も参考になるコメント

rowSpanを実行すると、機能します。 el.rowSpan = 2;実行するDOMインターフェイスとの一貫性を保つために、属性にはキャメルケースバージョンを使用します。 多くの人がこれに混乱するので、間違ったケースを使用したときに警告するために、私はすでに#255を開いています。

全てのコメント9件

rowSpanを実行すると、機能します。 el.rowSpan = 2;実行するDOMインターフェイスとの一貫性を保つために、属性にはキャメルケースバージョンを使用します。 多くの人がこれに混乱するので、間違ったケースを使用したときに警告するために、私はすでに#255を開いています。

ああ、それだけです。 ありがとう:)長期的には#255の解決を待ちますが、短期的にはコードを調整します...

申し訳ありませんが@ericclemmons 、それは可能性があります:) 0.4より前の欠落している属性のパスを実行しようとしましたが、 https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfigから欠落しているものに気付いた場合

ああ、 rowSpan (camelCased!)が欠落しているように見えます。 PRを提出します...

おっと! 私はcolSpanを見て、rowSpanもそこにあると素朴に想定しました…:)

@ericclemmonsまだこれを修正することに興味がありますか?

#291の締めくくり...

これがDOMとの一貫性を保つのではなく、人々をつまずかせることを考えると、HTMLとの一貫性を保ち、すべて小文字の属性をサポートしてみませんか?

「オートコンプリート」属性に関するこの問題のドキュメントに戻る必要がありました。

私はHTMLのように見えるものを書いています、それはHTMLのように機能するはずです...そうでなければ、それは一貫性ではなく、一貫性がありません。

同意する。 属性がすべて.toLowerCase()マップされていると、(私にとっては)望ましいでしょう。

このページは役に立ちましたか?
0 / 5 - 0 評価