<th rowspan="2"></th>
не устанавливает rowspan
в DOM.
Составная часть:
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>
...
Может это мой первый пиар? :)
Если вы сделаете rowSpan
это сработает - мы используем версию Camelcase для атрибутов для согласованности с интерфейсом DOM, где вы бы сделали el.rowSpan = 2;
. Я уже открыл # 255, чтобы предупредить о неправильном регистре, так как это многих сбивает с толку.
О, вот и все. Спасибо :) Я буду ждать разрешения # 255 надолго, но скорректирую свой код в ближайшее время ...
Извините @ericclemmons , это могло быть :) Я попытался выполнить пропуск для отсутствующих атрибутов до 0.4, но если вы заметили, что что-то отсутствует в https://github.com/facebook/react/blob/master/src/dom/DefaultDOMPropertyConfig .js , не стесняйтесь открывать выпуск или PR
О, похоже, что rowSpan
(camelCased!) Отсутствует . Я подам PR ...
Ой! Я увидел colSpan и наивно предположил, что rowSpan тоже будет там… :)
@ericclemmons Все еще
Закрытие на № 291 ...
Учитывая, что это сбивает людей с толку, вместо того, чтобы соответствовать DOM, почему бы не быть согласованным с HTML и не поддерживать атрибуты в нижнем регистре?
Мне просто пришлось вернуться к документации по этой проблеме об атрибуте «автозаполнение».
Я пишу что-то похожее на HTML, оно должно работать как HTML ... иначе это несогласованность, а не последовательность.
Я согласен. Было бы предпочтительнее (для меня), если бы все атрибуты были сопоставлены .toLowerCase()
.
Самый полезный комментарий
Если вы сделаете
rowSpan
это сработает - мы используем версию Camelcase для атрибутов для согласованности с интерфейсом DOM, где вы бы сделалиel.rowSpan = 2;
. Я уже открыл # 255, чтобы предупредить о неправильном регистре, так как это многих сбивает с толку.