Skip to content

Make My Space a root drop target#16

Merged
martinezharo merged 1 commit into
mainfrom
codex/add-drag-feature-to-move-elements
Jul 14, 2026
Merged

Make My Space a root drop target#16
martinezharo merged 1 commit into
mainfrom
codex/add-drag-feature-to-move-elements

Conversation

@martinezharo

Copy link
Copy Markdown
Owner

Motivation

  • Let users drop folders/snippets onto the sidebar "My Space" label to move or restore items to the workspace root, matching the existing root drop zone behaviour.

Description

  • Added isRootDropTarget and drag handlers on the My Space button in src/components/Aside/Aside.tsx to handle dragenter, dragover and drop and call the existing drag.enterDropTarget("root") / drag.dropOnFolder(null) flows.
  • Reused the existing root sentinel styling so the My Space label shows the same active drop state as the root drop area.

Testing

  • Ran TypeScript check with pnpm -s exec tsc --noEmit which succeeded.
  • Ran lint with pnpm -s lint which succeeded.
  • Ran unit tests with pnpm -s test (Vitest) and all tests passed (13 files, 141 tests).
  • Installed Playwright browsers and deps (pnpm -s exec playwright install chromium and pnpm -s exec playwright install-deps chromium) and captured a headless screenshot to verify the UI, which succeeded.

Codex Task

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
klipcode 507b844 Commit Preview URL

Branch Preview URL
Jul 14 2026, 11:34 AM

@martinezharo
martinezharo marked this pull request as ready for review July 14, 2026 11:37
@martinezharo
martinezharo merged commit ae3e66d into main Jul 14, 2026
3 checks passed
@martinezharo
martinezharo deleted the codex/add-drag-feature-to-move-elements branch July 14, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant