Skip to content
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

Implement simple move tool #24

Closed
wants to merge 1 commit into from
Closed

Conversation

gargsms
Copy link

@gargsms gargsms commented Jul 20, 2016

I had been experiencing troubles getting the text positioned exactly where I wanted it, so went ahead and implemented the move tool.

There is a shortcoming to this, however. Data is lost as the drawings move beyond the edge of the canvas.
Let me know if there is a workaround for that.

@viliusle
Copy link
Owner

viliusle commented Jul 21, 2016

First, thank you for trying make it better.

I checked your changes. But there are already similar tools:

  1. first tool "select tool" - can be used to move layer with mouse, or with keyboard arrows (shift, ctrl can be used too)
  2. second tool "select area tool" - part of layer can be selected and moved/transformed.

i will reject merge, let me know if something was missed.

p.s. i know issue with edges. System do not allow saving data outside, so for now it is normal behavior. Plus you can use it to erase layers left/right/top/bottom sides. Fixing it would be easy.

@gargsms
Copy link
Author

gargsms commented Jul 21, 2016

Now that you mention this, I realize that I completely overlooked this.
The PR feature is already there. Better I close this.

@gargsms gargsms closed this Jul 21, 2016
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.

None yet

2 participants