Page MenuHome
Create Task
Maniphest T46892

Snapping closest vertex with Pivot point set to Individual Origins
Closed, ResolvedPublic
Actions

Description

System Information
Win 10 64bit, GTX 970 (x2)

Blender Version
Broken: 2,76b official release
Worked: (optional)

Short description of error
If you set Pivot point to Individual Origins and you try to snap the closest vertex it does not work - instead it always uses the Median Snap Target(or maybe Center - I am not sure). I am not completely sure if this is a bug but IMHO it should snap to closest vertex no matter what pivot point do I use.

Exact steps for others to reproduce the error

Snap-Closest.blend457 KBDownload

  1. Open the attached blend
  2. In edit mode move the selected cube in x axis and try to snap it to a vertex of the other cube. It does not use the closest vertex but a median point instead.
  3. Solution: Change Pivot point to any other option and the snapping works as expected.

Revisions and Commits

rB Blender
rB21f376780941 fix T46892: snap to closest point now works with Individual Origins

Related Objects

Event Timeline

Marek Moravec (marek) created this task.Nov 28 2015, 8:45 PM
Marek Moravec (marek) raised the priority of this task from to 90.
Marek Moravec (marek) updated the task description. (Show Details)
Marek Moravec (marek) added a project: BF Blender.
Marek Moravec (marek) edited a custom field.
Marek Moravec (marek) added a subscriber: Marek Moravec (marek).
Campbell Barton (campbellbarton) lowered the priority of this task from 90 to 50.EditedNov 30 2015, 12:10 PM
Campbell Barton (campbellbarton) added a subscriber: Campbell Barton (campbellbarton).

Using the 'center' for snapping makes sense normally, but agree in this case its not ideal.

This is a general system though. So having exceptions needs to be handled carefully.

Possible fix, P295

Bastien Montagne (mont29) assigned this task to Campbell Barton (campbellbarton).Jan 22 2016, 3:34 PM
Bastien Montagne (mont29) added a subscriber: Bastien Montagne (mont29).
Campbell Barton (campbellbarton) removed Campbell Barton (campbellbarton) as the assignee of this task.Aug 3 2016, 6:34 AM
Campbell Barton (campbellbarton) removed a subscriber: Campbell Barton (campbellbarton).
Bastien Montagne (mont29) assigned this task to Germano Cavalcante (mano-wii).Aug 3 2016, 5:25 PM
Bastien Montagne (mont29) added a subscriber: Germano Cavalcante (mano-wii).

@Germano Cavalcante (mano-wii) maybe you’ll be interested by this one? You’re our snapping expert now. ;)

Germano Cavalcante (mano-wii) added a comment.Aug 4 2016, 3:44 PM
This comment was removed by Germano Cavalcante (mano-wii).
Germano Cavalcante (mano-wii) added a comment.Feb 3 2017, 4:04 PM

I don't see the need to calculate islands when translating. I propose to skip the island detection in this case

Possible fix 2, P444

Bastien Montagne (mont29) added a comment.Feb 3 2017, 4:29 PM

@Germano Cavalcante (mano-wii) Meh… don’t really have strong opinion here :/

I’d say, go by your solution, looks like it makes sense, and we can always revert and try something else if it creates other issues (pretty hard to predict what can cause what in that code).

Germano Cavalcante (mano-wii) added a comment.Feb 3 2017, 4:36 PM

Islands could be useful to identify the center of the groups when "Snap Target" is "Center" instead of "Closest".
But it does not work like that now and will not work that way after applying that "possible fix 2".

One solution to this case ( "Snap Target: Center") would be to preserve "island_info" (instead of freeing) and use its information to find the nearest center

Germano Cavalcante (mano-wii) changed the task status from Unknown Status to Resolved by committing rB21f376780941: fix T46892: snap to closest point now works with Individual Origins.Feb 3 2017, 5:16 PM
Germano Cavalcante (mano-wii) added a commit: rB21f376780941: fix T46892: snap to closest point now works with Individual Origins.
Campbell Barton (campbellbarton) mentioned this in rB3be073807bd7: Fix T51651: translate w/ individual origins fails.Jun 1 2017, 5:06 PM
Ankit Meel (ankitm) mentioned this in T79556: Snap vertices with "Closest" is not acting as expected.Aug 7 2020, 11:13 PM