[Bf-blender-cvs] [eb21222e64f] master: Transform: Disable snap to plane-face intersection

Germano Cavalcante noreply at git.blender.org
Wed Nov 4 19:23:52 CET 2020


Commit: eb21222e64f4333181d09b675b5505751ff2154c
Author: Germano Cavalcante
Date:   Wed Nov 4 15:22:01 2020 -0300
Branches: master
https://developer.blender.org/rBeb21222e64f4333181d09b675b5505751ff2154c

Transform: Disable snap to plane-face intersection

This feature was added in D5608.
But in practice this confuses more than it helps.
This fixes T82386.

===================================================================

M	source/blender/editors/transform/transform_constraints.c

===================================================================

diff --git a/source/blender/editors/transform/transform_constraints.c b/source/blender/editors/transform/transform_constraints.c
index de1c875a7b0..8b4b040b550 100644
--- a/source/blender/editors/transform/transform_constraints.c
+++ b/source/blender/editors/transform/transform_constraints.c
@@ -284,7 +284,7 @@ static void constraint_snap_plane_to_edge(const TransInfo *t, const float plane[
  * Snap to the nearest point between the snap point and the line that
  * intersects the face plane with the constraint plane.
  */
-static void constraint_snap_plane_to_face(const TransInfo *t, const float plane[4], float r_out[3])
+static void UNUSED_FUNCTION(constraint_snap_plane_to_face(const TransInfo *t, const float plane[4], float r_out[3]))
 {
   float face_plane[4], isect_orig[3], isect_dir[3];
   const float *face_snap_point = t->tsnap.snapPoint;
@@ -406,7 +406,8 @@ static void applyAxisConstraintVec(
             constraint_snap_plane_to_edge(t, plane, out);
           }
           else if (is_snap_to_face) {
-            constraint_snap_plane_to_face(t, plane, out);
+            /* Disabled, as it has not proven to be really useful. (See T82386). */
+            // constraint_snap_plane_to_face(t, plane, out);
           }
           else {
             /* View alignment correction. */



More information about the Bf-blender-cvs mailing list