Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow the QuickInput widget to be dragged #237534

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

TylerLeonhardt
Copy link
Member

@TylerLeonhardt TylerLeonhardt commented Jan 9, 2025

Thanks to @lszomoru for the initial prototype! That also had resizing but there were a couple edge cases that made it not quite ready.

I'm not sold on the grid lines. On macOS Spotlight just snaps, there's no grid. But let's keep it for now, it's easy to remove.

However, the drag-n-drop is really quite polished so I wanted to get this out for folks to try and I don't think it needs a setting.

Next steps:

  • resize
  • allow this to be synced with Settings Sync?

Fixes #17268

Thanks to @lszomoru for the initial prototype! That also had resizing but there were a couple edge cases that made it not quite ready.

However, the drag-n-drop is really quite polished so I wanted to get this out for folks to try and I don't think it needs a setting.

Next steps:
* Add a snap to the top (aka the original location)
* Resize...

Fixes #17268
@TylerLeonhardt TylerLeonhardt self-assigned this Jan 9, 2025
@TylerLeonhardt TylerLeonhardt enabled auto-merge (squash) January 9, 2025 00:24
@vs-code-engineering vs-code-engineering bot added this to the January 2025 milestone Jan 9, 2025
@TylerLeonhardt TylerLeonhardt merged commit 2327683 into main Jan 9, 2025
8 checks passed
@TylerLeonhardt TylerLeonhardt deleted the tyler/command-palette-drag branch January 9, 2025 04:42
@bpasero
Copy link
Member

bpasero commented Jan 9, 2025

Very cool 👏

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.

Make quick pick position configurable
4 participants