Skip to content

Commit

Permalink
Document fork status
Browse files Browse the repository at this point in the history
The README explains the justification of the fork.
Mention the issues for visibility from the original repo:

- Merge HubSpot/sortable#54
- Address HubSpot/sortable#46
  • Loading branch information
devoxel committed Oct 23, 2024
1 parent 9dcf54f commit 3329a5d
Showing 1 changed file with 20 additions and 6 deletions.
26 changes: 20 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,23 @@
## Sortable
## sortable-table

<a href="https://eager.io/app/sortable/install?source=button">
<img src="https://install.eager.io/install-button.png" border="0" width="126">
</a>
### Overview

##### [Demo](http://github.hubspot.com/sortable/docs/welcome) &nbsp;&nbsp; [Documentation](http://github.hubspot.com/sortable)
Sortable is an open-source JavaScript and CSS library which adds sorting
functionality to tables. It is tiny (`<2kb` min+gzip) and has no dependencies.

Sortable is an open-source JavaScript and CSS library which adds sorting functionality to tables. It is tiny (`<2kb` min+gzip) and has no dependencies.
### Fork Status

This is a fork of [HubSpot/sortable](https://github.com/HubSpot/sortable).
Due to a lack of maintenance, I felt it prudent to open a fork.

I will only look at PRs for critical / performance issues - I like the
simplicity of the original.

Changes include:

- Updates dev tooling to latest version (as of Oct 2024).
- Merged Hubspot/sortable#54 - "Fix performance issues with large tables". Thanks @hjacobs for writing this code.
- Addressed Hubspot/sortable#46 - "touch devices fires sorting twice". This if fixed to just use the touchstart event on mobile.
- Addressed a race with the DOM - I didn't file an issue for this one, but the script can run before DOM is ready, meaning the onclick events are not setup. I encoutered this when creating large tables and having the script in the `<head>`.

I will make this available on a CDN for easy access.

0 comments on commit 3329a5d

Please sign in to comment.