Skip to content

Conversation

mdtusz
Copy link
Contributor

@mdtusz mdtusz commented May 4, 2016

Replaces #502.

Fixes the drag interaction on annotation text and the arrow.

mdtusz added 2 commits May 4, 2016 11:47
When removing the last nested svg elements, annotations changed to use
translations for positioning. This fixes dragging when
`context.editable: true`.
@mdtusz mdtusz added bug something broken status: reviewable labels May 4, 2016
@etpinard
Copy link
Contributor

etpinard commented May 4, 2016

Looks good!

@coopy would you mind pulling down this branch and check if indeed it solves the issue you pointed out here ?

@coopy
Copy link

coopy commented May 4, 2016

It works like ✨ magic! ✨
Thank you, @mdtusz.

l9fqhzkvso

@etpinard
Copy link
Contributor

etpinard commented May 4, 2016

💃

expect(movedBox.left).toBe(elBox.left - 20);
expect(movedBox.top).toBe(elBox.top + 20);

mouseEvent('mouseup', elX - 20, elY + 20);
Copy link

Choose a reason for hiding this comment

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

Double ➕ good test!

@mdtusz mdtusz merged commit 5aef0e7 into master May 4, 2016
@mdtusz mdtusz deleted the annotation-dragging branch May 4, 2016 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants