← 输入框|github.com|刷新
Skip to content

web: stop cards dragging from overlays and fix drop placeholder pulse - #38

Merged
yekta merged 1 commit into
masterfrom
fix/disable-drag-under-overlays
Oct 17, 2026
Merged

yekta merged 1 commit into
masterfrom
fix/disable-drag-under-overlays

Conversation

@yekta

@yekta yekta commented Oct 17, 2026

Copy link
Copy Markdown
Contributor
  • Block drag behind overlays: the panels and dialogs a card opens are portaled out of the card, but React still bubbles their events up to it. Because of that, pressing inside an open sidepanel or modal could pick up and reorder cards in the background. The mouse, touch and keyboard sensors are now wrapped so a drag only starts when the press lands on the card itself (card-sorting.ts).
  • Fix the placeholder pulse on drop: the opacity transition on project and service cards was always on, so the dashed drop placeholder faded in and out when a card was released. The transition now only runs while a card is deleting, so a dropped card lands in its spot without flicker.
  • Consistent lifted shadow: on service cards with volumes, the volume rows covered the card's shadow, so some lifted cards looked like they had none. The shadow now sits on a wrapper around both the card and its volumes.
  • Larger lifted shadow: dragged project, service and service group cards now use shadow-xl instead of shadow-lg, so they look more clearly lifted.

- Panels and dialogs that a card opens are portaled out of the card, but React still bubbles their events up to it. That let presses inside an overlay pick up and reorder cards in the background. The sensors now only start a drag when the press lands on the card itself.
- The opacity transition was always on, so the dashed placeholder faded in and out on drop. It now only runs while a card is deleting.
- On service cards the shadow sits on a wrapper around the card and its volumes. The volumes were covering the shadow, so some lifted cards looked like they had none. Lifted cards also get a larger shadow, shadow-xl.
@yekta
yekta merged commit 3b83775 into master Oct 17, 2026
8 checks passed
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