Local Storage for Coffee

6 years after the last comment I needed this feature, as I wanted to offload this request to a multisite on with not a huge amount of resources.

Overall I feel it's quite good, I added an install hook, a configuration option and small theme tweaks so that you can see if the data is cached as well as an option to refresh it. See MR !8

We've had the patch from MR8 running for some time and is seeming good, currently applying to Drupal 10.2.7 with Coffee 8.x-1.4.
Help to performance so that the system is not unnecessarily scouring the site for all admin link options every time.
Thank you @hanoii

Bump this to Major regarding the improvement stated in #17 📌 Local Storage for Coffee Needs review "Help to performance so that the system is not unnecessarily scouring the site for all admin link options every time."

Maybe needs a reroll or new branch to sync MR with 📌 Automated Drupal 11 compatibility fixes for coffee RTBC for possible upcoming Coffee Drupal 11 release.

Just (re-)discovered this in 📌 Add performance testing Active .

Given that Coffee is activated by a link or keyboard shortcut, could the AJAX request not be deferred to then? If it's in local storage it should be instant, but it would help in the non-cached case and potentially also help with browser memory usage given the data structure wouldn't be loaded on most requests then.

On the approach here, I think it would make sense to key the local storage key by Drupal core version and/or $settings['deployment_identifier'] or possibly something added to Drupal settings that is based on a hash of the menus being fetched - could be written to state on menu link rebuild or similar. That would allow for a longer TTL then, or possibly even hard-coding the TTL if it's reliable enough. Core's toolbar module has similar logic to do a similar-ish thing.

Postponing this on 📌 Implement caching mechanism Active and 📌 Get coffee data only when the search box is opened Active - those can be done without complex complex cache invalidation logic (or cache staleness bugs), and should make this less necessary. If it's still necessary after those, then dynamic_page_cache support would allow for a short TTL here without much impact on server load.

📌 Get coffee data only when the search box is opened Active got in.

I am experiencing big performance issues when editing a node. The get-data coffee's call is blocking the loading of ckeditor for 2-3 seconds, what can I do ?

@yonailo can you confirm you're already on version 2.0.1 of coffee?

yes I have tested with 2.0.1

📌 Get coffee data only when the search box is opened Active should have changed things so that the request isn't made until the search box is opened. If that's running all the time, something else is broken. Do you have e.g. gin_toolbar enabled?

I will try to find out, non I don't have gin_toolbar my backoffice theme is claro.
Thank you for your return anyway.

I don't see any more the get-data call so i guess I have another issue, thank you.

This is so old, thanks for keeping the dream! My patches were for D7 so I'm hiding them.