style(web): polish the session sidebar - #1519
Conversation
…fic lights stay visible
# Conflicts: # apps/kimi-web/src/components/Sidebar.vue # apps/kimi-web/src/components/WorkspaceGroup.vue # apps/kimi-web/src/lib/icons.ts
…workspace row actions
🦋 Changeset detectedLatest commit: b46d2c0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
❌ Nix build failed Hash mismatch in
Please update |
commit: |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cd2ea73934
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Related Issue
No linked issue — this is a visual polish pass on the web sidebar (plus one small desktop fix), driven by direct product review rather than a ticket.
Problem
The session sidebar had drifted into a set of small but visible inconsistencies: cool-gray selection on a warm sidebar, a collapse toggle that flashed because two separate buttons handed off, misaligned session vs. workspace titles, workspace headers truncated by invisible hover buttons, and row heights imposed by
min-heightrather than the type. On the desktop app, the macOS traffic lights vanished when the window lost focus.What changed
Web sidebar (
apps/kimi-web):--color-sidebar-bg) and translucent neutral selection/hover tokens (--color-selected,--color-hover) so selection reads as location, not an accent action.Kbdkeycap primitive (one block per key); the search row now justifies the label and shortcut apart. Unified the separateShortcutKeyinto it.src/icons/kimi/) wired intounplugin-icons, plus Tabler icons for the panel toggle; sidebar row icons bumped to 16px.min-height); the hover kebab is absolutely positioned so it cannot force the row taller or cause jitter. Unified radius and padding.⋯and+) now float over the row instead of reserving layout space, so long workspace names use the full row width.--sb-guttermatches the folder icon width), muted workspace group titles, and aligned the workspace/session title font sizes.DesignSystemView.Desktop (
apps/kimi-desktop):Checklist
gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update. (No user-facing command/config changes; in-appDesignSystemViewkept in sync.)