Fix T55745: Pick Shortest Path tool, inconsistent selection.
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:
parent
887d6cb1ee
commit
bc675d90d0
Notes:
blender-bot
2023-02-14 05:38:34 +01:00
Referenced by issue #55745, Pick Shortest Path tool, inconsistent selection
|
@ -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) ||
|
||||
|
|
Loading…
Reference in New Issue