Page MenuHome

UI: Snapping Popover Labels
AbandonedPublic

Authored by Aaron Carlisle (Blendify) on Fri, Sep 6, 1:51 AM.

Details

Summary

Using the words element and target are more descriptive and are easier to describe to someone than using snap to/with.

Diff Detail

Event Timeline

I don't have a strong opinion on this.
Snap to and Snap with seem to deviate somewhat from the standard (which is using nouns).
Snap Element: and Snap Target: look more conventional and are also better than the originals changed in this patch D5242: Rename Snapping menu;

This revision is now accepted and ready to land.Fri, Sep 6, 4:38 AM

I don't think this regression is a good idea, considering the amount of support that the RightClickSelect proposal gathered in the first place, and the consensus around this in the recent patch that Germano mentioned.

That way, it looks like you're snap "Snap Element" to "Snap Target".

Read also "snap_target" tooltip: “Which part to snap onto the target”, i.e. target is the "snap_elements".

And also the "Align Rotation to Target".

Brecht Van Lommel (brecht) requested changes to this revision.EditedFri, Sep 6, 11:47 PM

I think the existing labels do a better job explaining what these do. "Target" to me implies the opposite of what it actually does, and modifiers and contraints also use it in the opposite way.

This revision now requires changes to proceed.Fri, Sep 6, 11:47 PM

Internally the logic of snap_elements and snap_target are a bit reversed which is what is really confusing. Reversing these makes sense:

Target (could also be called method)

The resulting point to snap the object to

Element

The part of the object to use for snapping.

The main goal of this patch as @Germano Cavalcante (mano-wii) pointed out is to follow the convention of using nouns for labels.

Conventions are fine, but we can deviate from them for the sake of clarity. I still think "with" explains what it does much better than "element".