Skip to content

fix(web): keep the sidebar resize handle above the chat composer background - #1766

Merged
kermanx merged 3 commits into
mainfrom
fix/web-sidebar-resize-handle
Jul 16, 2026
Merged

fix(web): keep the sidebar resize handle above the chat composer background#1766
kermanx merged 3 commits into
mainfrom
fix/web-sidebar-resize-handle

Conversation

@kermanx

@kermanx kermanx commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Related Issue

None — this is a clear, reproducible bug fix, so the problem is explained below per CONTRIBUTING.

Problem

In the web UI, hovering the sidebar resize strip only shows a sliver of the blue highlight: the chat composer's dock paints its opaque background over the half of the 4px strip that overhangs the conversation column, so the strip looks cut off and is hard to grab.

Root cause: both the composer dock (z-index: var(--z-sticky) = 100) and the resize handle (also 100) compete in the same stacking context, and the dock comes later in DOM order, so its background paints over the handle.

What changed

Raise the resize handle's z-index from --z-sticky (100) to --z-dropdown (200), so the full strip stays visible and draggable above pane-level sticky chrome (the composer dock and in-pane sticky headers at 100) while remaining under tooltips, overlays, and dropdown menus. One-line change in ResizeHandle.vue plus an explanatory comment.

Verified in a live browser against a local server: with a session open and the composer dock spanning the conversation column, the handle is the topmost element at its conversation-side half and the full blue bar renders on hover. typecheck, check:style, and oxlint pass for the web app.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works. (Pure CSS stacking change; the web package has no component-test infrastructure. Verified live in browser instead — see above.)
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update. (Bug fix restoring intended behavior; no documented behavior changes.)

@changeset-bot

changeset-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 82e7abe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

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

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 590b4984dc

ℹ️ 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".

z-index: var(--z-sticky);
/* above pane-level sticky chrome (chat dock, headers at --z-sticky): its 2px
overhang into the neighbour pane must stay visible and grabbable */
z-index: var(--z-dropdown);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Update the ResizeHandle design-system layer

With this handle now using --z-dropdown, the canonical design-system table still documents ResizeHandle as --z-sticky (apps/kimi-web/src/views/DesignSystemView.vue:1452), even though apps/kimi-web/AGENTS.md requires modified UI to match DesignSystemView.vue. That stale spec can mislead future UI changes back to the old stacking layer and reintroduce this regression, so please update the design-system entry alongside the component change.

Useful? React with 👍 / 👎.

@pkg-pr-new

pkg-pr-new Bot commented Jul 16, 2026

Copy link
Copy Markdown
pnpm dlx https://pkg.pr.new/@moonshot-ai/kimi-code@82e7abe
npx https://pkg.pr.new/@moonshot-ai/kimi-code@82e7abe

commit: 82e7abe

@kermanx
kermanx merged commit 7042af3 into main Jul 16, 2026
14 checks passed
@kermanx
kermanx deleted the fix/web-sidebar-resize-handle branch July 16, 2026 08:17
This was referenced Jul 16, 2026
ywh114 pushed a commit to ywh114/kimi-code that referenced this pull request Jul 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant