-
-
Notifications
You must be signed in to change notification settings - Fork 388
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
[feature request] Code minification #1044
Comments
Hiya @BenGamma, I mean files like 'scrollSpy.js' and ''custom.js'. UPDATE: I just used a online js code minifer to cut buttons.js by 7KiB as an example. I'd like to help in this effort if I can? Thank you for getting back to me. -J |
Oh and css files too! |
Ah you are refering to the academy. The repo for the academy is https://github.com/ONEARMY/academy We are using a tool called Docusaurus to build the academy section and we embed it in /academy. I never dive into the minified optimizations for it. I would say have a look if you like ! |
Hi @JanetLaysEggs - thanks for pointing this out. Yes, confusingly enough all the content for the academy sits in another git repo (and we just display it in an iframe in the community platform). I've copied your issue over to ONEARMY/academy#106 - if you did have a chance to look into ways we could improve on this then that would be great (although as a word of warning the repository isn't very actively maintained, so it could be some time for changes to be merged - but most welcome nonetheless) |
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
Currently it seems that the pages at https://community.preciousplastic.com/ use no code minification in any of the JS files. I recommend webpack or something similar to help with this and reduce server+ bandwidth load. Also what is being used for the back end for users etc?
Describe alternatives you've considered
The alternative is not using minifcation?
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: