UV: Edge support for select shortest path operator
ClosedPublic
Actions

Authored by Siddhartha Jejurkar (sidd017) on Jul 21 2022, 12:45 PM.

Details

Reviewers

Maniphest Tasks

T99344: Pick Shortest Path does not work as expected with edge select in some cases in UV editor

Summary

Calculating shortest path selection in UV edge mode was done using vertex
path logic. Since the UV editor now supports proper edge selection [0],
this approach can sometimes give incorrect results.

This problem is now fixed by adding separate logic to calculate the
shortest path in UV edge mode.

Resolves T99344.

[0]: ffaaa0bcbf477c30cf3665b9330bbbb767397169

Diff Detail

Repository

rB Blender

Branch

uv-edge-path-fix (branched from master)

Build Status

Buildable 23094
Build 23094: arc lint + arc unit

Event Timeline

Siddhartha Jejurkar (sidd017) requested review of this revision.Jul 21 2022, 12:45 PM

Siddhartha Jejurkar (sidd017) created this revision.

Harbormaster completed remote builds in B23094: Diff 53905.Jul 21 2022, 12:45 PM

Siddhartha Jejurkar (sidd017) added a task: T99344: Pick Shortest Path does not work as expected with edge select in some cases in UV editor.Jul 21 2022, 12:46 PM

Siddhartha Jejurkar (sidd017) added a reviewer: Campbell Barton (campbellbarton).

Siddhartha Jejurkar (sidd017) added a project: Restricted Project.

Add brackets (caused warning with GCC)

Harbormaster completed remote builds in B23103: Diff 53925.Jul 22 2022, 2:56 AM

Campbell Barton (campbellbarton) retitled this revision from Fix T99344: Edge support for UV shortest path operator to UV: Edge support for select shortest path operator.Jul 22 2022, 2:58 AM

Campbell Barton (campbellbarton) edited the summary of this revision. (Show Details)

Campbell Barton (campbellbarton) accepted this revision.Jul 22 2022, 3:01 AM

This revision is now accepted and ready to land.Jul 22 2022, 3:01 AM

Campbell Barton (campbellbarton) mentioned this in rB772574054373: UV: Edge support for select shortest path operator.Jul 22 2022, 3:54 AM

Committed rB77257405437336dbd91a481926013f8c747cacae thanks for working on this!