-
-
Notifications
You must be signed in to change notification settings - Fork 85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added spread support for relative and ordinal scopes #2254
Conversation
update from meet-up:
|
Updated! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just posting old pending comments; ignore for now
packages/common/src/types/command/PartialTargetDescriptor.types.ts
Outdated
Show resolved
Hide resolved
packages/cursorless-engine/src/processTargets/modifiers/targetSequenceUtils.ts
Outdated
Show resolved
Hide resolved
packages/cursorless-engine/src/processTargets/modifiers/targetSequenceUtils.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok made some minor tweaks. Also, it looks like your relative exclusive code is broken; it only uses the first and the last targets. I added some failing test cases so you can see what I mean. Fwiw I ran talon tests locally after my changes and they passed.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok should be good to go @AndreasArvidsson ! Was a good thing I updated cheatsheet—turns out it was broken due to the fact you factored simple_modifier
into two lists 😬
here are my changes in case helpful: https://github.com/cursorless-dev/cursorless/pull/2254/files/ac0bc761fda6621b7d7748fea98be34f90d86650..5ba078aff4cbc10d01508fd49a2f91899ecc727b
`"change every two tokens"` Fixes #1514 ## Checklist - [x] I have added [tests](https://www.cursorless.org/docs/contributing/test-case-recorder/) - [x] I have updated the [cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet) - [x] I have updated the [docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) - [x] I have not broken the cheatsheet --------- Co-authored-by: Pokey Rule <[email protected]>
"change every two tokens"
Fixes #1514
Checklist