Skip to content

Commit

Permalink
Fix move focus between grouped panes (#3045) (#3958)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Daniel599 authored and msftbot[bot] committed Dec 16, 2019
1 parent abb2df1 commit dccb297
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/cascadia/TerminalApp/Pane.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit dccb297

Please sign in to comment.