search

easysoft / zui

ZUI is an HTML5 front UI framework.
https://openzui.com
MIT License
2.71k stars 690 forks source link

对zui.datagrid.js的两处意见 #93

Closed zora-han closed 6 years ago

zora-han commented 6 years ago

根据在使用过程中出现的问题，进行的修改

catouse commented 6 years ago

第一个意见完全接受；第二个意见有如下问题： 索引仅仅是在界面上显示的顺序，无法永久的与某条数据绑定，有时候无法根据索引来区分数据，所以如果要启用行选中功能，最佳操作是为每条数据指定一个唯一的 ID（源码中的 rowId）。或者当 rowId 没有指定时退化为使用索引。

以后建议将每个功能改动提交一个单独的 Pull Request，这样便于跟踪和讨论。

zora-han commented 6 years ago

那也就是说，如此改动会不会更好一些？之前单纯用索引，确实是欠考虑了 config.rowId = (dataItem && dataItem.rowId)? dataItem.rowId : (rowIndex === 0 ? '#header' : rowIndex);

catouse commented 6 years ago

我的想法是这样：

var rowId = dataItem && (dataItem.rowId || dataItem.id);
config.rowId = rowId !== undefined ? rowId : (rowIndex === 0 ? '#header' : rowIndex);