Add open/close motion to quick pick#326032
Conversation
📬 CODENOTIFYThe following users are being notified based on files changed in this PR: @TylerLeonhardtMatched files:
|
There was a problem hiding this comment.
Pull request overview
Adds subtle open/close motion to the Quick Input (Quick Pick) widget when Modern UI (.style-override) is enabled and motion is enabled (.monaco-enable-motion). This aligns Quick Input behavior with other ephemeral surfaces that animate in/out.
Changes:
- Add CSS open/close animations for
.quick-input-widgetunder.style-override.monaco-enable-motion. - Update
QuickInputController.hide()to apply a “closing” class and deferdisplay: noneuntil the animation completes. - Add a browser test validating that motion is gated on Modern UI + motion enabled and that visibility state updates as expected.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/vs/platform/quickinput/test/browser/quickinput.test.ts | Adds coverage for close motion gating and timing. |
| src/vs/platform/quickinput/browser/quickInputController.ts | Implements close animation lifecycle and adjusts isVisible() semantics during close. |
| src/vs/platform/quickinput/browser/media/quickInput.css | Defines keyframes + selectors for open/close motion on the widget. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Screen.Recording.2026-07-15.at.5.03.46.PM.mov |
|
@hawkticehurst thank you! Looks nice in the case the palette is equal or wider to the background elements. I was just "worried" that those background elements disappeared before the palette spawned, causing a strange visual effect. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Related to #325020
Adds open/close motion to quick pick.
Screen.Recording.2026-07-15.at.3.06.24.PM.mov