-
Notifications
You must be signed in to change notification settings - Fork 8
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
💸 Megamenu - Bundle Size Too Large #1357
Comments
In optimizing the size of the package, could you please move those image assets to remote so we can utilize speed from cdn, etc? They take around 190kb(60%) space. |
Created a PR at SSWConsulting/SSW.MegaMenu#41 |
This was referenced Sep 11, 2023
SSW Rules issue: SSWConsulting/SSW.Rules#1097 |
Menu JSON to be merged when all tasks completed: SSWConsulting/SSW.Website.Menu.json#99 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Right now, the megamenu size is much too large - this is likely due to bundling npm depdendencies that could be classified as peerDependencies.
I have created an issue on SSW.Megamenu here: SSWConsulting/SSW.MegaMenu#40
Proposed solution
I will be moving as many NPM packages as possible to peer dependencies/removing the dependency where possible.
Tasks
The text was updated successfully, but these errors were encountered: