cursorless-dev / cursorless

Don't let the cursor slow you down
https://www.cursorless.org/
MIT License
1.09k stars 77 forks source link

Separate hat on/off functions #2462

Closed baltazarortiz closed 2 days ago

baltazarortiz commented 4 days ago

As per slack discussion, it would be great to have individual on/off commands in addition to toggle. Example usecase: if I'm presenting code I'd love to have hats on when talon is awake but turn them off when I sleep to discuss said code with coworkers who aren't used to ignoring colorful hats

AndreasArvidsson commented 3 days ago

Is it fine if it's the same command id and we just have an optional argument? I assume you're going to use the command server for this? user.run_rpc_command("cursorless.toggleDecorations", false)

baltazarortiz commented 2 days ago

I didn’t know that was an option but sounds great to me!

On Wed, Jul 3, 2024 at 5:21 AM Andreas Arvidsson @.***> wrote:

Is it fine if it's the same command id and we just have an optional argument? I assume you're going to use the command server for this? user.run_rpc_command("cursorless.toggleDecorations", false)

— Reply to this email directly, view it on GitHub https://github.com/cursorless-dev/cursorless/issues/2462#issuecomment-2205947162, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD4B2ZUF3R63VV3VHIPYGHLZKPUCRAVCNFSM6AAAAABKH5TFS6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBVHE2DOMJWGI . You are receiving this because you authored the thread.Message ID: @.***>

AndreasArvidsson commented 2 days ago

Pull request created https://github.com/cursorless-dev/cursorless/pull/2464