Skip to content

Fix(Regions): region can't be dragged to position 0 and duration.#4199

Merged
katspaugh merged 2 commits intokatspaugh:mainfrom
xiaoliyooo:main
Sep 23, 2025
Merged

Fix(Regions): region can't be dragged to position 0 and duration.#4199
katspaugh merged 2 commits intokatspaugh:mainfrom
xiaoliyooo:main

Conversation

@xiaoliyooo
Copy link
Collaborator

Short description

Resolves #

When dragging a region, can't be dragged to the start and end positions: 0 and duration.
There will be very little spacing

Implementation details

How to test it

Screenshots

Checklist

  • This PR is covered by e2e tests
  • It introduces no breaking API changes

@katspaugh katspaugh requested a review from Copilot September 22, 2025 14:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a bug where regions cannot be dragged to the start (position 0) or end (duration) positions, addressing spacing issues during region dragging.

  • Removed boundary checks that prevented dragging to position 0 and duration
  • Added explicit clamping to ensure regions stay within valid bounds

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Owner

@katspaugh katspaugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@xiaoliyooo
Copy link
Collaborator Author

Thank you!

Bro, you probably forgot to merge😀.

@katspaugh katspaugh merged commit 83210a8 into katspaugh:main Sep 23, 2025
4 checks passed
@katspaugh
Copy link
Owner

Merged :)

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.

3 participants