fix: fixes #79, cleanup follow logic when window is closed (#80)

Author: cseickel

lua/neo-tree/log.lua

@@ -86,8 +86,10 @@ log.new = function(config, standalone)
 
   obj.set_level = function(level)
     if levels[level] then
-      config.level = level
-      print("[neo-tree] Log level set to " .. level)
+      if config.level ~= level then
+        config.level = level
+        print("[neo-tree] Log level set to " .. level)
+      end
     else
       print("[neo-tree] Invalid log level: " .. level)
     end
@@ -108,6 +110,9 @@ log.new = function(config, standalone)
         x = tostring(round(x, config.float_precision))
       elseif type(x) == "table" then
         x = vim.inspect(x)
+        if #x > 200 then
+          x = x:sub(1, 200) .. "..."
+        end
       else
         x = tostring(x)
       end

lua/neo-tree/sources/filesystem/init.lua

@@ -146,19 +146,26 @@ M.float = function()
   M.navigate(state.path, path_to_reveal)
 end
 
-M.follow = function(callback)
+M.follow = function(callback, force_show)
   log.trace("follow called")
   local path_to_reveal = get_path_to_reveal()
   if not utils.truthy(path_to_reveal) then
-    return
+    return false
   end
   local state = get_state()
-  if not renderer.window_exists(state) then
-    return
+  if not force_show and not renderer.window_exists(state) then
+    return false
   end
   local is_in_path = path_to_reveal:sub(1, #state.path) == state.path
   if not is_in_path then
-    return
+    return false
+  end
+  local node = state.tree and state.tree:get_node()
+  if node then
+    if node:get_id() == path_to_reveal then
+      -- already focused
+      return false
+    end
   end
 
   log.debug("follow file: ", path_to_reveal)
@@ -208,6 +215,7 @@ M.follow = function(callback)
 
     events.subscribe(event)
   end)
+  return true
 end
 
 ---Focus the window, opening it if it is not already open.
@@ -251,12 +259,11 @@ local navigate_internal = function(path, path_to_reveal, callback)
     local follow_file = state.follow_current_file and get_path_to_reveal()
     if utils.truthy(follow_file) then
       M.follow(function()
-        renderer.focus_node(state, follow_file)
         if callback then
           callback()
         end
         state.in_navigate = false
-      end)
+      end, true)
     else
       local previously_focused = nil
       if state.tree and renderer.is_window_valid(state.winid) then
@@ -283,9 +290,6 @@ local navigate_internal = function(path, path_to_reveal, callback)
   end
 
   if path_changed then
-    if state.follow_current_file then
-      M.follow()
-    end
     if state.bind_to_cwd then
       vim.api.nvim_command("tcd " .. path)
     end

lua/neo-tree/ui/renderer.lua

@@ -198,10 +198,12 @@ M.focus_node = function(state, id, do_not_focus_window)
         if success then
           -- make sure we are not scrolled down if it can all fit on the screen
           local win_height = vim.api.nvim_win_get_height(state.winid)
-          if win_height > linenr then
-            vim.cmd("normal! zb")
-          elseif linenr < (win_height / 2) then
-            vim.cmd("normal! zz")
+          if vim.api.nvim_get_current_win() == state.winid then
+            if win_height > linenr then
+              vim.cmd("normal! zb")
+            elseif linenr < (win_height / 2) then
+              vim.cmd("normal! zz")
+            end
           end
         else
           log.warn("Failed to set cursor: " .. err)