Skip to content

Comments

Arrow bend translation#770

Open
trangdoan982 wants to merge 1 commit intomainfrom
cursor/ENG-1366-arrow-bend-translation-59c6
Open

Arrow bend translation#770
trangdoan982 wants to merge 1 commit intomainfrom
cursor/ENG-1366-arrow-bend-translation-59c6

Conversation

@trangdoan982
Copy link
Collaborator

@trangdoan982 trangdoan982 commented Feb 12, 2026

https://www.loom.com/share/49c3b4960dbb4baf8a5e1a7a2a1391e3

Fixes DiscourseRelationShape's onTranslate to prevent unexpected bend changes when moving multiple selected shapes.

Previously, when both ends of a DiscourseRelationShape were bound, its onTranslate method would always convert translation into bend changes. This was incorrect when the relation was moved as part of a larger selection of shapes (e.g., nodes and the connecting arrow). The change ensures that bend changes only occur if only the relation shape is selected, allowing for simple translation when moved with other shapes, aligning with tldraw's native arrow behavior.


Linear Issue: ENG-1366

Open in Cursor Open in Web


Open with Devin

- Check if only the relation shape is selected before applying bend changes
- When multiple shapes are selected together, perform simple translation
- Preserves existing behavior when dragging arrow alone (bend changes allowed)
- Aligns with tldraw native arrow behavior for group movements

Co-authored-by: Trang Doan <trangdoan982@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Feb 12, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@linear
Copy link

linear bot commented Feb 12, 2026

@supabase
Copy link

supabase bot commented Feb 12, 2026

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@trangdoan982 trangdoan982 marked this pull request as ready for review February 13, 2026 17:11
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

View 4 additional findings in Devin Review.

Open in Devin Review

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