Skip to content

Fix frozen drag size badge placement#57

Merged
yvette-carlisle merged 1 commit into
mainfrom
y/frozen-drag-size-badge-placement
Apr 4, 2026
Merged

Fix frozen drag size badge placement#57
yvette-carlisle merged 1 commit into
mainfrom
y/frozen-drag-size-badge-placement

Conversation

@yvette-carlisle
Copy link
Copy Markdown
Member

Summary

  • keep the shared size badge inside-fallback contract for live and window-frozen paths
  • add a drag-region-only outside-first fallback for frozen size badge placement when the default bottom toolbar slot overlaps
  • cover the drag-vs-window boundary with focused rendering behavior tests

Verification

  • cargo make fmt-rust
  • cargo test -p rsnap-overlay --lib selection_size_badge
  • cargo test -p rsnap-overlay overlay:: --lib
  • cargo make lint-rust
  • git diff --check

@yvette-carlisle
Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

…:"scope frozen size badge placement to drag-region sources","intent":"repair the PR head after review pointed out that the first patch rewrote the shared size-badge fallback contract instead of only aligning dragged frozen captures with the window-frozen placement behavior","impact":"FrozenCaptureSource::DragRegion now prefers an outside placement when the toolbar reserve overlaps the default slot while live and window-frozen paths keep the existing shared fallback contract, and regression tests now lock the drag-versus-window boundary","breaking":false,"risk":"low","authority":"review","delivery_mode":"status-only","refs":[]}
@yvette-carlisle yvette-carlisle force-pushed the y/frozen-drag-size-badge-placement branch from 4e3d191 to d822328 Compare April 4, 2026 14:52
@yvette-carlisle yvette-carlisle added this pull request to the merge queue Apr 4, 2026
Merged via the queue into main with commit 608d1e1 Apr 4, 2026
7 checks passed
@yvette-carlisle yvette-carlisle deleted the y/frozen-drag-size-badge-placement branch April 4, 2026 15:10
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