Skip to content

Make Mark Mode keybindings precede custom keybindings#13659

Merged
3 commits merged intomainfrom
dev/cazamor/mark-mode/mamo-precedes-kbd
Aug 22, 2022
Merged

Make Mark Mode keybindings precede custom keybindings#13659
3 commits merged intomainfrom
dev/cazamor/mark-mode/mamo-precedes-kbd

Conversation

@carlos-zamora
Copy link
Member

Summary of the Pull Request

This PR moves the key handling for mark mode into a helper method that is then called before an action/key binding is attempted.

References

Epic: #4993
Closes #13533

Validation Steps Performed

  • Add custom keybinding to "down" arrow key
  • in mark mode --> selection updates appropriately
  • out of mark mode --> keybinding executed

@ghost ghost added Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-1 A description (P1) labels Aug 2, 2022
@carlos-zamora carlos-zamora force-pushed the dev/cazamor/mark-mode/mamo-precedes-kbd branch from 83e41a9 to a8cd0cf Compare August 2, 2022 20:03
@carlos-zamora
Copy link
Member Author

Hmm... I'm marking this as a service-able bugfix. But if anybody disagrees, let me know.

@carlos-zamora carlos-zamora added this to the Terminal v1.16 milestone Aug 2, 2022
@carlos-zamora carlos-zamora force-pushed the dev/cazamor/mark-mode/mamo-precedes-kbd branch from 7f7de33 to d980227 Compare August 2, 2022 22:57
@zadjii-msft zadjii-msft added the AutoMerge Marked for automatic merge by the bot when requirements are met label Aug 22, 2022
@ghost
Copy link

ghost commented Aug 22, 2022

Hello @zadjii-msft!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 70313db into main Aug 22, 2022
@ghost ghost deleted the dev/cazamor/mark-mode/mamo-precedes-kbd branch August 22, 2022 20:39
DHowett pushed a commit that referenced this pull request Sep 6, 2022
This PR moves the key handling for mark mode into a helper method that is then called before an action/key binding is attempted.

Epic: #4993
Closes #13533

- Add custom keybinding to "down" arrow key
- in mark mode --> selection updates appropriately
- out of mark mode --> keybinding executed

Amended by DHowett to remove URL targeting features.

(cherry picked from commit 70313db)
Service-Card-Id: 84711521
Service-Version: 1.15
@ghost
Copy link

ghost commented Sep 13, 2022

🎉Windows Terminal v1.15.252 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link

ghost commented Sep 13, 2022

🎉Windows Terminal Preview v1.16.252 has been released which incorporates this pull request.:tada:

Handy links:

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-TerminalControl Issues pertaining to the terminal control (input, selection, keybindings, mouse interaction, etc.) AutoMerge Marked for automatic merge by the bot when requirements are met Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-1 A description (P1)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mark Mode should preempt key bindings, and work with anything that causes scrolling...

3 participants