Skip to content

Comments

Add feedback for more actions moving item edges, refactored cmdMoveIt…#1332

Merged
ScottChesworth merged 2 commits intojcsteh:masterfrom
ScottChesworth:feedbackMoveItemEdgesToCursor
Oct 29, 2025
Merged

Add feedback for more actions moving item edges, refactored cmdMoveIt…#1332
ScottChesworth merged 2 commits intojcsteh:masterfrom
ScottChesworth:feedbackMoveItemEdgesToCursor

Conversation

@ScottChesworth
Copy link
Collaborator

…emEdgeOrSource function to match codestyle

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@ScottChesworth
Copy link
Collaborator Author

I've added "Item edit: Move position of item to edit cursor" to the key map on Control+Alt+Shift+Comma Windows, Command+Option+Shift+Comma on Mac. It's replacing "Item edit: Move left edge of item to edit cursor" because this is a more intuitive behaviour paired up with the move right edge action. This way, both actions preserve item length. Cockos plan to clarify the name of the action we had mapped before, but they don't plan to change its behaviour, hence this key map adjustment.

@ScottChesworth ScottChesworth merged commit 5edf33b into jcsteh:master Oct 29, 2025
3 checks passed
@tbdalgaard
Copy link

This might break a few workflows like what @GarthHum does for editing. I have used this myself, and found it quite effective, because I can easily shrink an item to the cursor. Extremely useful for small edits, and ripple is not an issue in that regard. I can however see why you may want this change, here I would have used the nudge/set dialog instead, but if that helps more users then I might just use the custom actions we have for now, and remember the state of ripple. Just wanted to mention this.

@ScottChesworth
Copy link
Collaborator Author

Wouldn't trim left edge get the result you want in that context?

@tbdalgaard
Copy link

tbdalgaard commented Oct 30, 2025 via email

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.

2 participants