Affects move tool, path manipulation tool, shape tool & assistant tool.
Also add "alt" in the move tool for precision move.
BUG: 332526
rempt |
Krita |
Affects move tool, path manipulation tool, shape tool & assistant tool.
Also add "alt" in the move tool for precision move.
BUG: 332526
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Um, I must either be testing the wrong build or something, but for the move tool, the snap to axis still is on CTRL? I'm not sure what I'm doing wrong here...
Ah, got it. For, for instance the move tool, click, then press shift works, but I guess shift-then-click is more natural -- like it works for ctrl-click.
Okay, now the tool doesn't just ignore shift then click.
Turns out this was due to an undocumented feature. Ctrl+right click toggles between "move current layer" and "move layer with content." Who knew?
I'm sorry, I was away for most of the day, so I haven't actually re-tested your patch. If you're sure the move tool is fixed (and I guess you are...) then feel free to push: I won't be able to test it until Tuesday; tomorrow and Friday are day-job-days, this weekend my dad is visiting and Monday is a day-job day again :-(
So, go ahead and push if you're confident: the code is fine and the goal is fine, too.
Ah, I managed to find some more time tonight. Seems to me it works perfectly well now!
No big deal, just was away for the holiday weekend in the US and would have pushed a little while earlier!