-
Notifications
You must be signed in to change notification settings - Fork 24
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
Menu Block Cache #4
Comments
@kestraly Thanks for the report. I have added a block cache, please check again. |
So I found the best solution was to a specific megamenu cache. This reduced the load time of the menu on a large store from 6 seconds to 0.6 seconds once cached. Using this technique I can put in a pull request with this. The only issue is the loss of the is_active feature. Adding the is_active feature through JQuery might be a better solution, so that you can have a fully cached menu that loads instantly and then jQuery highlights the active category by adding into the menu li data-categoryid="CATEGORY_ID" example Then with jQuery something like that's on the page
|
Closing - Added Pull Request |
Judging by the timings. The menu is not stored in block cache?
The text was updated successfully, but these errors were encountered: