fix: avoid overwrite user defined hover function(#63)

Author: haorenW1025

after/plugin/completion.vim

@@ -0,0 +1 @@
+lua require'completion.hover'.modifyCallback()

lua/completion.lua

@@ -166,7 +166,6 @@ M.customize_buf_label = function(label)
 end
 
 M.on_attach = function()
-  hover.modifyCallback()
   api.nvim_command [[augroup CompletionCommand]]
     api.nvim_command("autocmd! * <buffer>")
     api.nvim_command("autocmd InsertEnter <buffer> lua require'completion'.on_InsertEnter()")

lua/completion/hover.lua

@@ -245,48 +245,47 @@ end
 -- Modify hover callback
 function M.modifyCallback()
   local callback = 'textDocument/hover'
+  local default_callback = vim.lsp.callbacks[callback]
   vim.lsp.callbacks[callback] = function(_, method, result)
     -- if M.winnr ~= nil and api.nvim_win_is_valid(M.winnr) then
       -- api.nvim_win_close(M.winnr, true)
     -- end
-    M.focusable_float(method, function()
-      if not (result and result.contents) then
-        -- return { 'No information available' }
-        return
-      end
-      local markdown_lines = vim.lsp.util.convert_input_to_markdown_lines(result.contents)
-      markdown_lines = vim.lsp.util.trim_empty_lines(markdown_lines)
-      if vim.tbl_isempty(markdown_lines) then
-        -- return { 'No information available' }
-        return
-      end
-      local bufnr, winnr
-      -- modified to open hover window align to popupmenu
-      if vim.fn.pumvisible() == 1 then
-        local position = vim.fn.pum_getpos()
-        -- Set max width option to avoid overlapping with popup menu
-        local total_column = api.nvim_get_option('columns')
-        local align
-        if position['col'] < total_column/2 then
-          align = 'right'
-        else
-          align = 'left'
+    if vim.fn.pumvisible() == 1 then
+      M.focusable_float(method, function()
+        if not (result and result.contents) then
+          -- return { 'No information available' }
+          return
         end
+        local markdown_lines = vim.lsp.util.convert_input_to_markdown_lines(result.contents)
+        markdown_lines = vim.lsp.util.trim_empty_lines(markdown_lines)
+        if vim.tbl_isempty(markdown_lines) then
+          -- return { 'No information available' }
+          return
+        end
+        local bufnr, winnr
+        -- modified to open hover window align to popupmenu
 
-        bufnr, winnr = fancy_floating_markdown(markdown_lines, {
-          pad_left = 1; pad_right = 1;
-          col = position['col']; width = position['width']; row = position['row']-1;
-          align = align
-        })
-        M.winnr = winnr
-      else
-        bufnr, winnr = vim.lsp.util.fancy_floating_markdown(markdown_lines, {
-          pad_left = 1; pad_right = 1;
-        })
-      end
-      vim.lsp.util.close_preview_autocmd({"CursorMoved", "BufHidden", "InsertCharPre"}, winnr)
-      return bufnr, winnr
-    end)
+          local position = vim.fn.pum_getpos()
+          -- Set max width option to avoid overlapping with popup menu
+          local total_column = api.nvim_get_option('columns')
+          local align
+          if position['col'] < total_column/2 then
+            align = 'right'
+          else
+            align = 'left'
+          end
+          bufnr, winnr = fancy_floating_markdown(markdown_lines, {
+            pad_left = 1; pad_right = 1;
+            col = position['col']; width = position['width']; row = position['row']-1;
+            align = align
+          })
+          M.winnr = winnr
+        vim.lsp.util.close_preview_autocmd({"CursorMoved", "BufHidden", "InsertCharPre"}, winnr)
+        return bufnr, winnr
+      end)
+    else
+      default_callback(_, method, result, _)
+    end
   end
 end