optimization: Enable short-circuiting for zoom check in should_tmux_c…

…ontrol

This change cuts out the `tmux` CLI invocation when
`disable_nav_when_zoomed = false`.

This reordering was necessary to enable this workaround because the
`and` operator is short-circuiting, and thus it will only evaluate the
RHS if the LHS is `true`. Before, the slow function call was the LHS,
and so it was being evaluated unconditionally, even when the
significantly faster boolean check would have failed.

README.md

@@ -232,6 +232,13 @@ more:
    possible workaround using `pgrep` (this might fail to work in some cases
    though; if you do find such a case please open an issue).
 
+   If you are still having performance problems when switching panes inside of
+   Neovim, consider setting `disable_when_zoomed = false` in the configuration
+   for this plugin. When enabled, this config option causes
+   `nvim-tmux-navigation` to invoke the `tmux` CLI every time you switch panes;
+   disabling this switch circumvents this system call and makes switching panes
+   much faster!
+
 3. Q: The plugin doesn't work when interacting with
    [Poetry](https://python-poetry.org/) shells.
    A: This happens because Poetry spawns sub-tty's, therefore messing with

lua/nvim-tmux-navigation/tmux_util.lua

@@ -32,7 +32,7 @@ end
 
 -- whether tmux should take control over the navigation
 function util.should_tmux_control(is_same_winnr, disable_nav_when_zoomed)
-    if is_tmux_pane_zoomed() and disable_nav_when_zoomed then
+    if disable_nav_when_zoomed and is_tmux_pane_zoomed() then
         return false
     end
     return is_same_winnr