Skip to content

Add shortcut badges to top sidebar navigation links#1986

Merged
jbecke merged 1 commit intomainfrom
codex/add-shortcut-labels-to-top-sidebar
Mar 16, 2026
Merged

Add shortcut badges to top sidebar navigation links#1986
jbecke merged 1 commit intomainfrom
codex/add-shortcut-labels-to-top-sidebar

Conversation

@jbecke
Copy link
Copy Markdown
Contributor

@jbecke jbecke commented Mar 16, 2026

Motivation

  • Make the top navigation rows (Inbox, Search, Agents, etc.) visually consistent with the bottom action buttons by surfacing their keyboard shortcuts inline.
  • Help users discover navigation hotkeys by showing the same styled shortcut chips used elsewhere in the sidebar.

Description

  • Added a shortcut badge to each SidebarLink render in js/app/packages/app/component/app-sidebar/sidebar.tsx that displays either g + key for go-to scoped links or a single key for standalone shortcuts.
  • The new chips reuse the existing Hotkey component and match the visual style used in the bottom action section.
  • Chips respect existing slim-sidebar behavior via the group-data-[slim=true]/sidebar:invisible class so they hide when the sidebar is collapsed.
  • Kept existing click/navigation behavior and hotkey registration intact; this change only augments the UI for each link.

Testing

  • Ran bun run check which failed in this environment due to missing local type definition packages (errors for @types/facebook-pixel, vite/client, etc.).
  • Ran lint for the modified file via the repo lint script which failed here due to registry access issues when resolving biome (npm 403 during fetch).
  • Attempted a Playwright snapshot against http://localhost:3000/app/component/unified-list but the local dev server was not available (ERR_EMPTY_RESPONSE), so no UI screenshot could be captured.

Codex Task

@jbecke jbecke requested a review from a team as a code owner March 16, 2026 23:30
@github-actions
Copy link
Copy Markdown

@jbecke jbecke merged commit 2c98655 into main Mar 16, 2026
23 checks passed
@jbecke jbecke deleted the codex/add-shortcut-labels-to-top-sidebar branch March 16, 2026 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant