Skip to content

Compass rose translation gizmo in the Select tool's transform cage #2282

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

Closed
Keavon opened this issue Feb 11, 2025 · 1 comment · Fixed by #2277, #2296 or #2298
Closed

Compass rose translation gizmo in the Select tool's transform cage #2282

Keavon opened this issue Feb 11, 2025 · 1 comment · Fixed by #2277, #2296 or #2298
Assignees
Labels
Good First Issue Good for newcomers
Milestone

Comments

@Keavon
Copy link
Member

Keavon commented Feb 11, 2025

Image

  1. The rotation/scaling grip/handle squares should rotate with the layer, not with the viewport. Notice how the layer has a local rotation and the grips use that rotation.
  2. We want to put a ring in the center that can be dragged to translate the transform cage. This is important for non-filled shapes which have no selection area inside them. In the image below, you'll see the click target in translucent blue. That light blue should appear when it or the solid ring is hovered. (Inside the ring is not a click target though, since it needs to give space for the pivot widget to be draggable.)
  3. That ring should have local-axis constrained drag arrows. Either side of the arrow (left/right; or up/down) is equivalent. Similar functionality to G X X or G Y Y but without actually being in G/R/S mode and no seeing the number overlays, etc. But we do want an overlay that is just a dashed line through the dragged arrow points going through the entire viewport. It should b either red or green to match the arrow being dragged. Collectively we call the ring and arrows the "compass rose" if you want to use that for your variable names.

Image

Image

Discussion in Discord here with exact specs and other details and chatter.


Possible follow-on work that's somewhat related: #2225

@Keavon Keavon added the Good First Issue Good for newcomers label Feb 11, 2025
@github-project-automation github-project-automation bot moved this to Short-Term in Task Board Feb 11, 2025
@mTvare6
Copy link
Contributor

mTvare6 commented Feb 11, 2025

Will be fixed by #2277

@github-project-automation github-project-automation bot moved this from Short-Term to Completed This Milestone in Task Board Feb 15, 2025
Keavon added a commit that referenced this issue Feb 20, 2025
)

* Fix mouse states and priority order of operation

* Add metadata for tampered transform

* Add comments explaining details

* Improve comments

* Move out of bounds checks into rotate check

---------

Co-authored-by: Keavon Chambers <[email protected]>
@Keavon Keavon added this to the Milestone 45 milestone Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment