fix(layout): always do instant resize when views were added or deleted

lua/edgy/animate.lua

@@ -136,6 +136,7 @@ function M.schedule()
         M.schedule()
       else
         M.restore_views(nil, true)
+        Util.debug("animation complete")
         Config.animate.on_end()
       end
     end, 1000 / Config.animate.fps)

lua/edgy/layout.lua

@@ -117,6 +117,7 @@ local function restore_state()
   end
 end
 
+---@return boolean changed whether the layout changed
 local function update()
   ---@type table<string, number[]>
   local wins = {}
@@ -128,23 +129,30 @@ local function update()
       table.insert(wins[ft], win)
     end
   end
+
+  local changed = false
   -- Update the windows in each sidebar
   M.foreach({ "bottom", "top", "left", "right" }, function(sidebar)
-    sidebar:update(wins)
+    if sidebar:update(wins) then
+      changed = true
+    end
   end)
+  return changed
 end
 
 ---@param opts? {full: boolean}
 function M.layout(opts)
   opts = opts or {}
 
-  update()
+  local changed = update()
+  local needs_layout = M.needs_layout()
 
-  if opts.full and not M.needs_layout() then
+  if opts.full and not (changed or needs_layout) then
     return false
   end
 
-  if opts.full then
+  if opts.full and needs_layout then
+    Util.debug("full layout")
     M.foreach({ "bottom", "top", "left", "right" }, function(sidebar)
       sidebar:layout()
     end)

lua/edgy/sidebar.lua

@@ -117,10 +117,15 @@ function M:on_hide(win)
 end
 
 ---@param wins table<string, number[]>
+---@return boolean updated if the sidebar was updated
 function M:update(wins)
   self.visible = 0
+  local updated = false
   for _, view in ipairs(self.views) do
-    self.visible = self.visible + view:update(wins[view.ft] or {})
+    if view:update(wins[view.ft] or {}) then
+      updated = true
+    end
+    self.visible = self.visible + #view.wins
     wins[view.ft] = vim.tbl_filter(function(w)
       for _, win in ipairs(view.wins) do
         if win.win == w then
@@ -131,6 +136,7 @@ function M:update(wins)
     end, wins[view.ft] or {})
   end
   self:_update({ check = true })
+  return updated
 end
 
 ---@param opts? {check: boolean}

lua/edgy/util.lua

@@ -26,6 +26,16 @@ function M.warn(msg)
   M.notify(msg, vim.log.levels.WARN)
 end
 
+function M.info(msg)
+  M.notify(msg, vim.log.levels.INFO)
+end
+
+function M.debug(msg)
+  if require("edgy.config").debug then
+    M.info(msg)
+  end
+end
+
 ---@generic F: fun()
 ---@param fn F
 ---@param max_retries? number

lua/edgy/view.lua

@@ -33,20 +33,22 @@ function M.new(opts, sidebar)
 end
 
 ---@param wins window[]
+---@return boolean updated Whether the view was updated
 function M:update(wins)
   ---@type table<window, Edgy.Window>
   local index = {}
   for _, w in ipairs(self.wins) do
     index[w.win] = w
   end
+  local old = self.wins
   self.wins = {}
   for _, win in ipairs(wins) do
     local buf = vim.api.nvim_win_get_buf(win)
     if not self.filter or self.filter(buf, win) then
       self.wins[#self.wins + 1] = index[win] or Window.new(win, self)
     end
   end
-  return #self.wins
+  return not vim.deep_equal(old, self.wins)
 end
 
 ---@param opts? {check: boolean}