From dccb29790eeb78b2ab48248eb91b68b5f5c4e99e Mon Sep 17 00:00:00 2001 From: Daniel599 Date: Tue, 17 Dec 2019 01:17:06 +0200 Subject: [PATCH] Fix move focus between grouped panes (#3045) (#3958) This PR fixes the ability to move between already-split panes ## Summary of the Pull Request ## PR Checklist * [x] Closes #3045 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Requires documentation to be updated * [ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx ## Detailed Description of the Pull Request / Additional comments ## Validation Steps Performed Tested manually. --- src/cascadia/TerminalApp/Pane.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cascadia/TerminalApp/Pane.cpp b/src/cascadia/TerminalApp/Pane.cpp index 967633cba4..5a53cb7fc5 100644 --- a/src/cascadia/TerminalApp/Pane.cpp +++ b/src/cascadia/TerminalApp/Pane.cpp @@ -230,7 +230,7 @@ bool Pane::_NavigateFocus(const Direction& direction) // If the child we want to move focus to is _already_ focused, return false, // to try and let our parent figure it out. - if (newlyFocusedChild->WasLastFocused()) + if (newlyFocusedChild->_HasFocusedChild()) { return false; }