microsoft/Terminal

`Pane::_FindFocusAndNeighbor` can probably be simpler

Open

#10,748 建立於 2021年7月21日

在 GitHub 查看
 (0 留言) (1 反應) (0 負責人)C++ (3,212 fork)batch import
Area-CodeHealthHelp WantedIssue-TaskProduct-Terminal

倉庫指標

Star
 (35,764 star)
PR 合併指標
 (平均合併 27天 19小時) (30 天內合併 24 個 PR)

描述

If we view this problem from the point of the focused pane, what this function (and the other) do is basically:

  • move to the parent node
  • check if the other child node - the one the focused pane doesn't belong to - is in the direction we want to move the pane to
    • if yes: swap panes
    • if no: continue with the parent node

Is that correct? If my understanding is correct, I believe we can simplify this code after it has been merged.

Originally posted by @lhecker in https://github.com/microsoft/terminal/pull/10638#discussion_r674151375

貢獻者指南