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

Fix T103624: Last node is skipped for cursor and link picking

Browse Source 
I wrote the reverse iteration incorrectly in e091291b5b.
This commit is contained in:
Hans Goudey
2023-01-04 10:06:10 -05:00
parent 22fec7b1a4
commit 20b2d6fc71
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
source/blender/editors/space_node/node_draw.cc
View File

@@ -2601,7 +2601,7 @@ static const bNode *find_node_under_cursor(SpaceNode &snode, const float2 &curso
{
/* Check nodes front to back. */
const Span<bNode *> nodes = snode.edittree->all_nodes();
for (int i = nodes.index_range().last(); i > 0; i--) {
for (int i = nodes.index_range().last(); i >= 0; i--) {
if (BLI_rctf_isect_pt(&nodes[i]->runtime->totr, cursor[0], cursor[1])) {
return nodes[i];
}

2
source/blender/editors/space_node/node_edit.cc
View File

@@ -1130,7 +1130,7 @@ bNodeSocket *node_find_indicated_socket(SpaceNode &snode,
snode.edittree->ensure_topology_cache();
const Span<bNode *> nodes = snode.edittree->all_nodes();
for (int i = nodes.index_range().last(); i > 0; i--) {
for (int i = nodes.index_range().last(); i >= 0; i--) {
bNode &node = *nodes[i];
BLI_rctf_init_pt_radius(&rect, cursor, size_sock_padded);