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

Align 2d Cursor #116

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BenjaminSauder
Copy link
Contributor

This patch extends the align operator a bit - such that one can easily snap the 2d cursor or switch the align mode temporarily.

This allows for a faster and more fluid workflow, as one doesnt need to fiddle around with the dropdown menu, or change selections to place the cursor onto a vertex.

Code changes should be easy to follow, nothing complicated going on :)

@franMarz
Copy link
Owner

The cursor mode of our Align utility refers to aligning the selection to the cursor position, not aligning the cursor itself. We can extend the utility so we can do both, but that is not what you are doing here neither I like the lack of arrangement to clearly divide one method from the other.

@BenjaminSauder
Copy link
Contributor Author

BenjaminSauder commented Dec 11, 2021

I agree the mix-up with the cursor mode is pretty confusing.

I'm not sure what's the best approach to make it more clear.

I would avoid adding much UI for this as it just adds more clutter, which should be avoided...as it's quite big already with the 3x4 buttons.

Maybe it could be solved with better names?

Another idea would be to introduce it as a new 'mode' - but I really want to keep the swiftness of the modifier keys ... But there are only three.

Anyhow the current workflow with switching modes via the dropdown is very slow and not really fluent, modifier keys allow for a much smoother experience here.

I really think that selection and canvas are way more important than the slightly obscure cursor mode. And placing the cursor to scale some uvs from a pivot is needed regularly.

@franMarz
Copy link
Owner

Maybe it could be solved with better names?

"to Cursor" instead of "Cursor" for the former functionality, "Cursor" for the one you're proposing. The problem is: cursor to canvas, or to selection bounding box? Both are useful, but the naming... "Cursor to sel" and "Cursor to can"? Pretty ugly. And yes, having two modes (UVs, Cursor) would be too much from an UI perspective, unfortunately. Maybe we can manage this just via modifier keys? Obscure, but is something.

Anyhow the current workflow with switching modes via the dropdown is very slow and not really fluent, modifier keys allow for a much smoother experience here.

I think that part is a really good addition, but first we should solve the problem of arrangement and final set of modes commented above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

Successfully merging this pull request may close these issues.

2 participants