Skip to content

refactor(menu): cleaned up menu#341

Merged
ShakiraPortfolio merged 7 commits intomainfrom
jh-ui-menu
Mar 12, 2026
Merged

refactor(menu): cleaned up menu#341
ShakiraPortfolio merged 7 commits intomainfrom
jh-ui-menu

Conversation

@johannehouweling
Copy link
Collaborator

@johannehouweling johannehouweling commented Mar 12, 2026

Fixes #339
Fixes #338
Fixes #340
Fixes #343
Fixes #345

Also adds caching to speed up external data decencies by more than factor 10.

Cleaned up the menu a lot:

image

Cleaner Search
image

Paging for Tools, automatic parsing of tools from ...service.json, addresses #338
image

Full visibility for methods:
image

Restored balance of mobile, addresses #339
image

by shifting Search into off-canvas on mobile and cleaned up dropdown menus on off-canvas:
image

Now also includes methods in search (methods and tools are pulled directly from methods_index.json and service_index.json, no longer hardcoded, addresses #340
image

@johannehouweling
Copy link
Collaborator Author

Now also includes methods in search (methods and tools are pulled directly from methods_index.json and service_index.json, no longer hardcoded, addresses #340

See above

@johannehouweling
Copy link
Collaborator Author

Removed toplevel location from search to address #345.

@johannehouweling
Copy link
Collaborator Author

johannehouweling commented Mar 12, 2026

@ozancinar, @egonw I added caching to Flask, to cache external interactions (loading of "service_index.json" and querying Zenodo and BioStudies) which are slow moving data repos.

comes with a new dependency flask-caching, which I added to requirements.txt

Caching now for 5days. Makes sense to review if other locations can make use of this. Tried to refactor aleady, but still might be some locations left that do a request directly to the .json. But the longest time are the Zenodo and BioStudy API calls which are now entirely cached. For the data tab this reduces loading times by a magnitude from ~2s to 0.2s (locally).

w/o chache
image

w cache on second reload
image

@johannehouweling
Copy link
Collaborator Author

Rebased this branch to make mergeable.

@ShakiraPortfolio ShakiraPortfolio merged commit 3ee6d7f into main Mar 12, 2026
2 checks passed
@ShakiraPortfolio ShakiraPortfolio removed their request for review March 12, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment