Skip to content

Annotation dragging #505

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 4, 2016
Merged

Annotation dragging #505

merged 2 commits into from
May 4, 2016

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