Skip to content

use a rubber band selection for moving or deleting multiple waypoints - #3183

Closed
ReimarBauer wants to merge 2 commits into
Open-MSS:developfrom
ReimarBauer:i1576
Closed

use a rubber band selection for moving or deleting multiple waypoints#3183
ReimarBauer wants to merge 2 commits into
Open-MSS:developfrom
ReimarBauer:i1576

Conversation

@ReimarBauer

Copy link
Copy Markdown
Member

Purpose of PR?:

Fixes #1576

mutilselection_example.mp4

with support from claude

@ReimarBauer
ReimarBauer requested a review from joernu76 September 9, 2026 16:57
@joernu76

Copy link
Copy Markdown
Member

Nice, but this introduces 329 lines-of-code in an already questionable part of the code (this broke a lot in the past) for making it slightly more comfortable to work with (I admit, there are some use cases). In light of keeping things manageable I propose to reject both the MR and the issue.

@ReimarBauer

Copy link
Copy Markdown
Member Author

We can reopen this issue when we plan to refactor that section of the path interactor. I previously attempted a manual implementation, but we should aim to avoid a situation where a model becomes a necessity for this specific area of the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

move or delete several waypoints in one step

2 participants