jadrake75 / ng-scrolling-table

MIT License
8 stars 2 forks source link

using row index is insufficient #44

Closed jadrake75 closed 10 years ago

jadrake75 commented 10 years ago

The selection mixin is using a row index which is insufficient if filtering (such as sorting of table filtering) is enabled. Should use an ID or reference to drive the id selected.

Default row id should be "ref-id" (property). This should be overrideable in the stgScrollingTable via a "data-row-ref-id" property.

If no id is present on the model, a generated id could be used? id should be treated as a string (even though it could be a number) so it can handle a uuid or other form.

this would allow functions listening for the "selection" event to look for the model with a matching id.