Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

(README) Note minimal Jquery UI dependencies #154

Closed
ivandotv opened this issue Apr 9, 2014 · 6 comments · Fixed by #158
Closed

(README) Note minimal Jquery UI dependencies #154

ivandotv opened this issue Apr 9, 2014 · 6 comments · Fixed by #158

Comments

@ivandotv
Copy link

ivandotv commented Apr 9, 2014

Jquery UI - sortable has these dependencies declared:

  • UI Core
  • Widget Factory
  • Mouse Interaction

I'm I correct in assuming that I could create a custom build of Jquery UI with only those dependencies and have no issues?

@thgreasi
Copy link
Contributor

thgreasi commented Apr 9, 2014

As far as I know those 3 should be enough, I never tried it though.
I would like to know your findings. It would be nice if you also attached a custom build url like:

http://jqueryui.com/download/#!version=1.10.4&components=1110000010000000000000000000000000

@ivandotv
Copy link
Author

Tested it, everything should work correctly. Here is the download link to only dependencies selected.
JQuery UI sortable dependencies
It slims down jQuery UI from ~230kb to ~38kb.

@thgreasi
Copy link
Contributor

That's nice!
I will run all the unit tests with the minimal build and as soon as all of them pass, we should better add a section to readme.MD.
Thanks for sharing your findings.

@thgreasi
Copy link
Contributor

I just created a branch and all tests seem to pass. I will close this as soon as README.md gets updated.

@thgreasi thgreasi changed the title Minimal set of dependencies for Jquery UI (README) Minimal set of dependencies for Jquery UI Apr 13, 2014
@thgreasi thgreasi changed the title (README) Minimal set of dependencies for Jquery UI (README) Note minimal Jquery UI dependencies Apr 13, 2014
@ivandotv
Copy link
Author

Thanks.

@thgreasi
Copy link
Contributor

I just tested the minified file sizes using the jsdelivr cdn.
I got 12.2KB vs 72.8KB for jquery-ui 1.10.x .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants