knaeckekami / diffutil_sliverlist

Hi there 👋

diffutil_sliverlist's Issues

Index in builder function

I'd like to have access to an item's index in the builder function, in addition to the existing BuildContext and item T. If needed, I could offer this change myself in a PR. Let me know!

Implementation of move is missing

Hi!

Great work with this package! I have a question regarding the missing move-implementation, is there a reason for this? It seems to be supported in diff_util?

void _onDiffUpdate(diffutil.DiffUpdate update) {
  update.when<void>(
      move: (_, __) =>
          throw UnimplementedError('moves are currently not supported'),
      insert: _onInserted,
      change: _onChanged,
      remove: _onRemoved);
}

Best regards

Handling/copying the lists provided to DiffUtilSliverList

The documentation states:

Don't mutate the list passed into DiffUtilSliverList yourself
...
either copy the list before mutation:
...
Or just copy the list before passing it to DiffUtilSliverList:
DiffUtilSliverList<int>(items: List.from(list), ...),

Is there a reason why this library does not automatically copy the list for users so this isn't needed? The list will need to be copied at some point for DiffUtil to work correctly anyways; I don't see why DiffUtilSliverList can't copy the list itself and take some work away from library users. Plus, this addition won't be a breaking change.

If I am missing something here/not understanding this correctly, please do let me know. But, if not, I'd be happy to open a PR for this.