react-spectrum icon indicating copy to clipboard operation
react-spectrum copied to clipboard

(WIP) Dnd api simplification

Open LFDanLu opened this issue 2 years ago โ€ข 10 comments

Closes

โœ… Pull Request Checklist:

  • [ ] Included link to corresponding React Spectrum GitHub Issue.
  • [ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • [ ] Filled out test instructions.
  • [ ] Updated documentation (if it already exists for this component).
  • [ ] Looked at the Accessibility Practices for this feature - Aria Practices

๐Ÿ“ Test Instructions:

๐Ÿงข Your Project:

LFDanLu avatar Jun 24 '22 00:06 LFDanLu

TODOs:

  • add tests for the new drag type/File/Directory filtering we've added to defaultOnDrop
  • Address https://github.com/adobe/react-spectrum/pull/3266#discussion_r964190132
  • add controls to the ListView story so we can turn on/off the util handlers
  • Research how other DnD libraries handle deleting items from a list on drop
    • This will determine onItemRemove's final place in the api (i.e. how do we communicate what dragged items were actually processed by the drop handlers)
  • other todos sprinkled throughout the code
  • fix tests in 16/17

LFDanLu avatar Sep 07 '22 00:09 LFDanLu