[Dev UI] [New Nav] Show / hide empty runners

firebase / genkit

An open source framework for building AI-powered apps with familiar code-centric patterns. Genkit makes it easy to develop, integrate, and test AI features with observability and evaluations. Genkit works with various models and platforms.

Apache License 2.0

783 stars 118 forks source link

[Dev UI] [New Nav] Show / hide empty runners #1344

Open mjchristy opened 1 week ago

mjchristy commented 1 week ago

Considerations

By default, we always show Models, Flows and Prompts - as these are key building blocks of genkit
TBD - If Empty (i.e. all menu items have (0)), show them all?

Copy: "View more" "View less"

How will this interact with runtime connection? We should persist their nav state unless they refresh.

Screenshots

Collapsed State

Annotations:

MichaelDoyle commented 1 week ago

Re: persistence, currently we do a "hard refresh" when the connection is re-established. We probably need to revisit that behavior in a more general sense.

tonybaroneee commented 1 week ago

@hugomurillomtz FYI - there's already some preexisting, unused logic in nav-toggle.service.ts that you can leverage here. Check out the areRunnersCollapsed and saveRunnerCollapseState helper methods, which use localStorage to remember the collapsed/expanded state.