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

draw: constrain object move with Shift #7958

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hfiguiere
Copy link
Contributor

@hfiguiere hfiguiere commented Jan 5, 2024

Issue #7933

Change-Id: Ie39c15a8552276bc5919ffbbd5dd859695048b9c

Summary

Press shift when dragging (screencast out of date)

Screencast.from.2024-01-05.17-49-59.webm

TODO

  • cleanup log
  • implement 45 deg constrain like on desktop
  • split out the geometry functions
  • add a threshold on the directional change (see screencast)
  • fix the issue where the move seems to not complete (see screencast)
  • test other apps: writer and calc
  • fix Draw: Select+Drag make drag without feedback #7821 (separate issue)

Checklist

  • Code is properly formatted
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@hfiguiere hfiguiere force-pushed the private/hub/constrain-drag branch from bea298c to 0e64b3a Compare January 8, 2024 22:48
@hfiguiere
Copy link
Contributor Author

Currently sitting on top of #7860 - it's needed.

@hfiguiere hfiguiere force-pushed the private/hub/constrain-drag branch from 0e64b3a to e869091 Compare January 9, 2024 15:20
@Minion3665 Minion3665 force-pushed the private/hub/constrain-drag branch from e869091 to 10c1365 Compare January 11, 2024 09:44
@Minion3665
Copy link
Member

(rebased as #7860 was merged)

@hfiguiere hfiguiere force-pushed the private/hub/constrain-drag branch from 10c1365 to 731a51a Compare January 11, 2024 12:03
@hfiguiere hfiguiere force-pushed the private/hub/constrain-drag branch 2 times, most recently from 849c195 to e7134e3 Compare January 23, 2024 23:47
@hfiguiere hfiguiere force-pushed the private/hub/constrain-drag branch from e7134e3 to a42e8af Compare January 24, 2024 14:00
@hfiguiere hfiguiere marked this pull request as ready for review January 24, 2024 16:28
@hfiguiere hfiguiere marked this pull request as draft January 25, 2024 13:46
@hfiguiere hfiguiere force-pushed the private/hub/constrain-drag branch from a42e8af to 724ff3e Compare March 14, 2024 13:10
@gokaysatir gokaysatir removed their request for review March 26, 2024 10:33
- Added src/layer/vector/Path.Constraint.ts

Issue #7933

Signed-off-by: Hubert Figuière <[email protected]>
Change-Id: Ie39c15a8552276bc5919ffbbd5dd859695048b9c
@eszkadev
Copy link
Contributor

@gokaysatir is this PR something stil valid with new grid?

@gokaysatir
Copy link
Contributor

@gokaysatir is this PR something stil valid with new grid?

Yes this seems still good to have. Just checked the feature, not the code.

@hfiguiere
Copy link
Contributor Author

It was imperfect back then and some recent rework broke it completely. The part that did constrain when dragging is gone as SVGGroup got refactored. I didn't try to fix it since.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: To Review
Development

Successfully merging this pull request may close these issues.

Draw: Constrained move Draw: Select+Drag make drag without feedback
4 participants