Skip to content

Conversation

@Mark-Phillipson
Copy link
Contributor

This pull request adds new voice commands to the text_navigation.talon file that allow users to select multiple items using the "items" keyword. These commands provide an alternative way to select text by specifying a quantity and direction.

New selection commands:

  • Added <number_small> items: command to select multiple items downward using shift-down and repeat for the specified count.
  • Added left <number_small> items: and right <number_small> items: commands to select multiple items to the left or right using shift-left and shift-right, repeating for the specified count.

Copilot AI review requested due to automatic review settings December 14, 2025 13:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds three new voice commands to enable selecting text using the "items" keyword, providing an alternative selection interface based on quantity and direction. These commands use simple keyboard shortcuts (shift + arrow keys) with repetition to perform the selection.

Key Changes

  • Added <number_small> items: command to select multiple items downward
  • Added left <number_small> items: and right <number_small> items: commands for horizontal selection

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

# Select Using the Items Keyword
<number_small> items:
key(shift-down)
repeat(number_small - 1)
Copy link

Copilot AI Dec 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spacing around the minus operator is inconsistent with the codebase style. Other uses of repeat in the repository use spaces around the minus operator, such as in core/modes/dictation_mode.talon. This should be changed to repeat(number_small - 1) for consistency.

Copilot uses AI. Check for mistakes.
repeat(number_small - 1)
left <number_small> items:
key(shift-left)
repeat(number_small - 1)
Copy link

Copilot AI Dec 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spacing around the minus operator is inconsistent with the codebase style. Other uses of repeat in the repository use spaces around the minus operator, such as in core/modes/dictation_mode.talon. This should be changed to repeat(number_small - 1) for consistency.

Copilot uses AI. Check for mistakes.
repeat(number_small - 1)
right <number_small> items:
key(shift-right)
repeat(number_small - 1)
Copy link

Copilot AI Dec 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The spacing around the minus operator is inconsistent with the codebase style. Other uses of repeat in the repository use spaces around the minus operator, such as in core/modes/dictation_mode.talon. This should be changed to repeat(number_small - 1) for consistency.

Copilot uses AI. Check for mistakes.
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.

1 participant