It would be good to split the `html5sortable.ts` file into smaller modules to make it easier to test them and understand the code. Important would be: - [ ] try to avoid using globals as much as possible (always inject arguments in functions rather tan relying on options) - [ ] try to include typescript definitions for the functions - [ ] try to include tests for the new parts - [ ] try to see if we can generalize a function so that it can be reused for other parts - [ ] include comments and documentations in the module files - [ ] try to make the function naming convention more straight forward, logical and homogeneous Good candidates are: - [findSortable](https://github.com/lukasoppermann/html5sortable/blob/master/src/html5sortable.ts#L116) - [findDragElement](https://github.com/lukasoppermann/html5sortable/blob/master/src/html5sortable.ts#L131) - [_getDragging](https://github.com/lukasoppermann/html5sortable/blob/master/src/html5sortable.ts#L81) & rename to make clear what the difference between findDragElement and `getDragging` is
It would be good to split the
html5sortable.tsfile into smaller modules to make it easier to test them and understand the code.Important would be:
Good candidates are:
getDraggingis