scottohara / tvmanager

PWA for tracking recorded, watched & upcoming TV shows
MIT License
3 stars 0 forks source link

Remove jQuery #79

Closed scottohara closed 2 years ago

scottohara commented 5 years ago

Over the years, the web platform has added new features such as

We are now at the point where a lot of the reasons we used jQuery initially are no longer relevant, and we should consider removing this dependency in favour of using the native platform equivalents.

The benefit is one less dependency, which should save around ~30kb (gzipped) - ~85kb (uncompressed); as well as one less global (and the need to use webpack.ProvidePlugin({jQuery: "jquery"}) in our bundles).

Before we can remove jQuery, however, we need to find a replacement for jQuery UI Sortable (https://github.com/scottohara/tvmanager/issues/78)

scottohara commented 4 years ago

As part of the migration to IndexedDb, we replaced $.ajax() calls with fetch() calls.