Skip to content
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

Merged
merged 17 commits into from
Mar 25, 2024
Merged

Conversation

AndreasArvidsson
Copy link
Member

@AndreasArvidsson AndreasArvidsson commented Feb 27, 2024

"change every two tokens"

Fixes #1514

Checklist

  • I have added tests
  • I have updated the cheatsheet
  • I have updated the docs
  • I have not broken the cheatsheet

@pokey
Copy link
Member

pokey commented Feb 27, 2024

update from meet-up:

  • switch spoken form to "every two tokens". In the future, if we want every other token, that would be "every second token", and every pair of tokens would be "two tokens every second token"
  • switch internal identifier to isEvery

@AndreasArvidsson
Copy link
Member Author

Updated!

Copy link
Member

@pokey pokey left a 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

changelog/2024-03-addedSpreadModifier.md Outdated Show resolved Hide resolved
changelog/2024-03-addedSpreadModifier.md Outdated Show resolved Hide resolved
docs/user/README.md Outdated Show resolved Hide resolved
cursorless-talon/src/spoken_forms.json Outdated Show resolved Hide resolved
Copy link
Member

@pokey pokey left a 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.

@pokey pokey self-assigned this Mar 17, 2024
@pokey
Copy link
Member

pokey commented Mar 24, 2024

  • update cheatsheet

Copy link
Member

@pokey pokey left a 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

@AndreasArvidsson AndreasArvidsson added this pull request to the merge queue Mar 25, 2024
Merged via the queue into main with commit 463a00f Mar 25, 2024
14 checks passed
@AndreasArvidsson AndreasArvidsson deleted the spread branch March 25, 2024 19:16
cursorless-bot pushed a commit that referenced this pull request Mar 25, 2024
`"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]>
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.

Create "spread" version of relative / ordinal modifiers
2 participants