Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bundle JS files #189

Closed
gohai opened this issue Feb 16, 2015 · 4 comments
Closed

Bundle JS files #189

gohai opened this issue Feb 16, 2015 · 4 comments

Comments

@gohai
Copy link

gohai commented Feb 16, 2015

Right now on master, loading the media page of an object fires 43 HTTP GET requests to load various JavaScript files, and I believe this will be the same for other pages in providence. I'd love to have a straightforward way to bundle these files into a single, minified file to improve page load time.

I see that the develop branch did away with the app/conf/javascript.conf file, so not sure if this is still relevant. But if you can give me a idea how to do this cleanly, I'd love to give it a shot in master.

@stefankeidel
Copy link

The javascript.conf is now called assets.conf and is still there :-)

I agree that it's a lot, I'm not sure if it's a giant problem though. Once these files have been loaded once, the web server responds with a 304 on every one of the asset requests and the browser loads it from cache. The only one that goes out with actual data is the page itself.

Any potential feature pull requests should be based off develop. As far as the actual implementation goes, maybe Seth can comment on that.

@collectiveaccess
Copy link
Owner

I'd love to add bundling and minification. The assets.conf (formerly javascript.conf) file did not start out quite so large and has just grown and grown. There are a number of js libraries referenced in it that are no longer even used (or loaded), and the time is ripe for a cleanup. Any changes should be done in the develop branch though.

If you want to take a shot at working on this we'd welcome you. If you just want it fixed then create a JIRA please. Thanks!

@kehh
Copy link
Contributor

kehh commented Aug 19, 2015

OK this got inadvertantly closed by me cherry-picking a commit from our fork (https://github.com/wamuseum/providence. The commit above (8eb9197) doesn't relate to this current issue but wamuseum#189

@stefankeidel
Copy link

I worked this a little bit in the https://github.com/collectiveaccess/providence/tree/dev/reorganizeAssets branch a while back. I should revive that code at some point :-(

@stefankeidel stefankeidel reopened this Aug 19, 2015
leftclickben pushed a commit to leftclickben/providence that referenced this issue May 23, 2016
* list item and storage locations in hierarchy browsers
* Closes collectiveaccess#189
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants