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

Add keybinding docs for tabSwitcher #107

Merged
merged 3 commits into from
Aug 20, 2020
Merged

Add keybinding docs for tabSwitcher #107

merged 3 commits into from
Aug 20, 2020

Conversation

leonMSFT
Copy link
Collaborator

@leonMSFT leonMSFT commented Aug 10, 2020

Keybinding docs for the Advanced Tab Switcher - microsoft/terminal#6732
This should probably not go into master but into one of the release branches 😄

edit: still need to update after 7321 goes in

@opbld31
Copy link

opbld31 commented Aug 10, 2020

Docs Build status updates of commit 8c86e5a:

⚠️ Validation status: warnings

File Status Preview URL Details
TerminalDocs/new-terminal-arguments.md ⚠️Warning View Details
TerminalDocs/customize-settings/key-bindings.md ✅Succeeded View

TerminalDocs/new-terminal-arguments.md

  • Line 11, Column 1: [Warning-multiple-h1s] Multiple H1s are not allowed. You can only have one top-level heading.
  • Line 11, Column 1: [Suggestion-duplicate-h1s] H1 'New Terminal Arguments in the Windows Terminal' is duplicated with other articles: 'new-terminal-arguments.md(11,1), new-terminal-arguments.md(11,1)'. First level headings must be unique within a docset. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@carlos-zamora
Copy link
Contributor

I think this PR is out of sync with microsoft/terminal#7321

Copy link
Contributor

@DHowett DHowett left a comment

Choose a reason for hiding this comment

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

yep, blokk

@opbld32
Copy link

opbld32 commented Aug 19, 2020

Docs Build status updates of commit b7b4ae6:

⚠️ Validation status: warnings

File Status Preview URL Details
TerminalDocs/new-terminal-arguments.md ⚠️Warning View Details
TerminalDocs/customize-settings/global-settings.md ✅Succeeded View
TerminalDocs/customize-settings/key-bindings.md ✅Succeeded View

TerminalDocs/new-terminal-arguments.md

  • Line 11, Column 1: [Warning-multiple-h1s] Multiple H1s(H1 'New Terminal Arguments in the Windows Terminal'(panes.md(98,1) -> new-terminal-arguments.md(11,1))) are not allowed. You can only have one top-level heading.
  • Line 11, Column 1: [Suggestion-duplicate-h1s] H1 'New Terminal Arguments in the Windows Terminal' is duplicated with other articles: 'new-terminal-arguments.md(11,1), new-terminal-arguments.md(11,1)'. First level headings must be unique within a docset. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@opbld31
Copy link

opbld31 commented Aug 19, 2020

Docs Build status updates of commit c0062cb:

⚠️ Validation status: warnings

File Status Preview URL Details
TerminalDocs/new-terminal-arguments.md ⚠️Warning View Details
TerminalDocs/customize-settings/global-settings.md ✅Succeeded View
TerminalDocs/customize-settings/key-bindings.md ✅Succeeded View

TerminalDocs/new-terminal-arguments.md

  • Line 11, Column 1: [Warning-multiple-h1s] Multiple H1s(H1 'New Terminal Arguments in the Windows Terminal'(panes.md(98,1) -> new-terminal-arguments.md(11,1))) are not allowed. You can only have one top-level heading.
  • Line 11, Column 1: [Suggestion-duplicate-h1s] H1 'New Terminal Arguments in the Windows Terminal' is duplicated with other articles: 'new-terminal-arguments.md(11,1), new-terminal-arguments.md(11,1)'. First level headings must be unique within a docset. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.

If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

For any questions, please:

@cinnamon-msft cinnamon-msft merged commit ffbaa45 into master Aug 20, 2020
@cinnamon-msft cinnamon-msft deleted the dev/lelian/ats branch August 20, 2020 22:50
ghost pushed a commit to microsoft/terminal that referenced this pull request Aug 21, 2020
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? -->
## Summary of the Pull Request
This PR splits the anchored and unanchored tab switcher into two. The anchored tab switcher is now baked into `nextTab`/`prevTab`, and the unanchored tab switcher command is just named `tabSearch`. `tabSearch` takes no arguments. To reflect this distinction, `CommandPalette.cpp` now refers to one as `TabSwitchMode` and the other as `TabSearchMode`.

I've added a global setting named `useTabSwitcher` (name up for debate) that makes the Terminal use the anchored tab switcher experience for `nextTab` and `prevTab`. 

I've also given the control the ability to detect <kbd>Alt</kbd> KeyUp events and to dispatch keybinding events. By listening for keybindings, the ATS can react to `nextTab`/`prevTab` invocations for navigation in addition to listening for <kbd>tab</kbd> and the arrow keys.

Closes #7178 
<!-- Please review the items on the PR checklist before submitting-->
## PR Checklist
* [x] CLA signed.
* [x] Documentation updates: MicrosoftDocs/terminal#107
* [x] Schema updated.
donno2048 added a commit to donno2048/terminal that referenced this pull request Sep 28, 2020
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? -->
## Summary of the Pull Request
This PR splits the anchored and unanchored tab switcher into two. The anchored tab switcher is now baked into `nextTab`/`prevTab`, and the unanchored tab switcher command is just named `tabSearch`. `tabSearch` takes no arguments. To reflect this distinction, `CommandPalette.cpp` now refers to one as `TabSwitchMode` and the other as `TabSearchMode`.

I've added a global setting named `useTabSwitcher` (name up for debate) that makes the Terminal use the anchored tab switcher experience for `nextTab` and `prevTab`. 

I've also given the control the ability to detect <kbd>Alt</kbd> KeyUp events and to dispatch keybinding events. By listening for keybindings, the ATS can react to `nextTab`/`prevTab` invocations for navigation in addition to listening for <kbd>tab</kbd> and the arrow keys.

Closes #7178 
<!-- Please review the items on the PR checklist before submitting-->
## PR Checklist
* [x] CLA signed.
* [x] Documentation updates: MicrosoftDocs/terminal#107
* [x] Schema updated.
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.

6 participants