Support configuring popup kind

Author: saghen

README.md

@@ -245,7 +245,7 @@ neogit.setup {
     kind = "floating_console",
   },
   popup = {
-    kind = "split",
+    kind = "popup",
   },
   stash = {
     kind = "tab",

doc/neogit.txt

@@ -259,7 +259,7 @@ to Neovim users.
         kind = "floating_console",
     },
     popup = {
-        kind = "split",
+        kind = "popup",
     },
     stash = {
         kind = "tab",

lua/neogit/config.lua

@@ -94,6 +94,7 @@ end
 ---| "vsplit" Open in a vertical split
 ---| "floating" Open in a floating window
 ---| "floating_console" Open in a floating window across the bottom of the screen
+---| "popup" Open in a popup
 ---| "auto" vsplit if window would have 80 cols, otherwise split
 
 ---@class NeogitCommitBufferConfig Commit buffer options
@@ -518,7 +519,7 @@ function M.get_default_values()
       kind = "floating_console",
     },
     popup = {
-      kind = "split",
+      kind = "popup",
     },
     stash = {
       kind = "tab",
@@ -776,13 +777,14 @@ function M.validate_config()
         "floating",
         "floating_console",
         "replace",
+        "popup",
         "auto",
       }, val)
     then
       err(
         name,
         string.format(
-          "Expected `%s` to be one of 'split', 'vsplit', 'split_above', 'vsplit_left', tab', 'floating', 'replace' or 'auto', got '%s'",
+          "Expected `%s` to be one of 'split', 'vsplit', 'split_above', 'split_above_all', 'split_below', 'split_below_all', 'vsplit_left', tab', 'floating', 'floating_console', 'replace', 'popup' or 'auto', got '%s'",
           name,
           val
         )

lua/neogit/lib/popup/init.lua

@@ -1,3 +1,4 @@
+local config = require("neogit.config")
 local PopupBuilder = require("neogit.lib.popup.builder")
 local Buffer = require("neogit.lib.buffer")
 local logger = require("neogit.logger")
@@ -410,7 +411,7 @@ function M:show()
   self.buffer = Buffer.create {
     name = self.state.name,
     filetype = "NeogitPopup",
-    kind = "popup",
+    kind = config.values.popup.kind,
     mappings = self:mappings(),
     status_column = " ",
     autocmds = {