cursorless icon indicating copy to clipboard operation
cursorless copied to clipboard

Cursorless tutorial ced

Open saidelike opened this issue 1 year ago • 4 comments

Currently requires https://github.com/saidelike/talon_hud/tree/cursorless fork of the talon HUD

Checklist

  • [ ] Fix scope tree view
  • [ ] Fix broken references to css eg reset
  • [ ] Merge HUD changes
  • [ ] Properly support custom actions and other necessary custom spoken forms
  • [ ] Properly support custom symbol spoken forms (eg alphabet)
  • [ ] Figure out how to properly detect VSCode focused
  • [ ] Tune tutorial content? Maybe better for follow-up PR
  • [ ] Highlight marks that need to be targeted, so user doesn't have to search for them
  • [ ] Remove print statements and commented out code from tutorial.py
  • [ ] Figure out how to only run step preparation when you have the right window focused. Some of the time @pokey gets error message after "cursorles help" step image
  • [ ] Open new window on start?
  • [ ] Ensure you're in the right window / editor before restoring state, so we don't accidentally clobber something
  • [ ] I have added tests
  • [ ] I have updated the docs and cheatsheet
  • [ ] I have not broken the cheatsheet

saidelike avatar Dec 13 '23 19:12 saidelike

@pokey I've pushed some very nice changes that make things being parsed from the extension side. Not everything is implemented but I would be curious if the general idea is fine with you.

saidelike avatar Jan 02 '24 17:01 saidelike

An initial version of the tutorial is live. Please use https://github.com/saidelike/talon_hud/tree/cursorless fork/branch for the HUD

saidelike avatar Jan 05 '24 17:01 saidelike

this PR seems to have pulled in some extraneous commits, so it's really hard to review 😅. If you're not sure how to clean it up, lmk and I can help you

pokey avatar Jan 10 '24 15:01 pokey

this PR seems to have pulled in some extraneous commits, so it's really hard to review 😅. If you're not sure how to clean it up, lmk and I can help you

It is because you had to merge the main branch to fix some issues iirc. You should be able to diff against that commit from the main branch maybe?

saidelike avatar Jan 10 '24 21:01 saidelike

closing in favour of the original https://github.com/cursorless-dev/cursorless/pull/360, as it's easier to use a branch that's part of the main repo

pokey avatar Apr 22 '24 14:04 pokey