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

Add grid placement tool #26313

Merged
merged 52 commits into from
Oct 8, 2024

Let right-click reset grid to default values

de2f9de
Select commit
Loading
Failed to load commit list.
Merged

Add grid placement tool #26313

Let right-click reset grid to default values
de2f9de
Select commit
Loading
Failed to load commit list.
Task list completed / task-list-completed succeeded Oct 8, 2024 in 0s

4 / 4 tasks completed

All tasks have been completed

Details

Required Tasks

Task Status
#26309 Completed
#26310 Completed
#26311 Completed
#29986 Completed
I click and drag on the playfield from point A to somewhere near point B Incomplete
Interaction ends Incomplete
I start the interaction again Incomplete
Draggable handle appears in a different location far away from point B due to spacing subdivision Incomplete
Seek to point B on the timeline Incomplete
Drag the handle a huge distance to point B Incomplete
Manually end interaction Incomplete
2 left clicks for setting origin, spacing and rotation Incomplete
2 left clicks in the same location for setting origin Incomplete
1 right click for setting origin Incomplete
1 drag for setting origin, spacing, and rotation Incomplete
2 left clicks for setting origin, spacing and rotation Incomplete
2 left clicks in the same location for setting origin Incomplete
1 right click for setting origin Incomplete
1 drag for setting origin, spacing, and rotation Incomplete
I don't think the right-click method needs to exist. You can just left-click twice to get the same behaviour right? Incomplete
I preferred this being on the left, as it clears the user's selection and behaves like a modal operation but now has no indication that it will do that. Can we move it back? Incomplete
It seems to maintain a spacing under 64 while the limit is 128, is there a reason for that? Incomplete