griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix T55745: Pick Shortest Path tool, inconsistent selection.

Browse Source 
To get consistent, user-expected results here, we need to 'fake'
starting immediately after a 'skip' block (such that we start with a
full block of selected elements).

Same issue affected vertices and edges selection of course, did not
check the other usages of WM_operator_properties_checker_interval_test()
though.
This commit is contained in:
Bastien Montagne
2018-08-13 17:40:44 +02:00
parent 887d6cb1ee
commit bc675d90d0
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
source/blender/editors/mesh/editmesh_path.c
View File

@@ -176,7 +176,8 @@ static void mouse_mesh_shortest_path_vert(
}
} while ((node = node->next));
int depth = 1;
/* We need to start as if just *after* a 'skip' block... */
int depth = op_params->interval_params.skip;
node = path;
do {
if ((is_path_ordered == false) ||
@@ -366,7 +367,8 @@ static void mouse_mesh_shortest_path_edge(
}
} while ((node = node->next));
int depth = 1;
/* We need to start as if just *after* a 'skip' block... */
int depth = op_params->interval_params.skip;
node = path;
do {
if ((is_path_ordered == false) ||
@@ -511,7 +513,8 @@ static void mouse_mesh_shortest_path_face(
}
} while ((node = node->next));
int depth = 1;
/* We need to start as if just *after* a 'skip' block... */
int depth = op_params->interval_params.skip;
node = path;
do {
if ((is_path_ordered == false) ||