Affects move tool, path manipulation tool, shape tool & assistant tool.
Also add "alt" in the move tool for precision move.
BUG: 332526
Details
Details
- Reviewers
rempt - Group Reviewers
Krita - Commits
- R37:d78edfd404d5: Configure shift key = snap to axis in all tools
Diff Detail
Diff Detail
- Repository
- R37 Krita
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
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...
Comment Actions
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.
Comment Actions
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?
Comment Actions
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.
Comment Actions
Ah, I managed to find some more time tonight. Seems to me it works perfectly well now!
Comment Actions
No big deal, just was away for the holiday weekend in the US and would have pushed a little while earlier!