From 5957e7a29005c286eaf2cc619e826cfd980eb5ff Mon Sep 17 00:00:00 2001 From: William Boman Date: Sat, 25 Dec 2021 21:47:57 +0100 Subject: feat(ui): sort servers alphabetically, also add language hints (#361) --- lua/nvim-lsp-installer/ui/status-win/init.lua | 32 +++++++++++++---- .../ui/status-win/server_hints.lua | 41 ++++++++++++++++++++++ 2 files changed, 66 insertions(+), 7 deletions(-) create mode 100644 lua/nvim-lsp-installer/ui/status-win/server_hints.lua (limited to 'lua') diff --git a/lua/nvim-lsp-installer/ui/status-win/init.lua b/lua/nvim-lsp-installer/ui/status-win/init.lua index 3191c6ae..4bc5de4d 100644 --- a/lua/nvim-lsp-installer/ui/status-win/init.lua +++ b/lua/nvim-lsp-installer/ui/status-win/init.lua @@ -5,6 +5,7 @@ local Data = require "nvim-lsp-installer.data" local display = require "nvim-lsp-installer.ui.display" local settings = require "nvim-lsp-installer.settings" local lsp_servers = require "nvim-lsp-installer.servers" +local ServerHints = require "nvim-lsp-installer.ui.status-win.server_hints" local HELP_KEYMAP = "?" local CLOSE_WINDOW_KEYMAP_1 = "" @@ -307,7 +308,9 @@ end ---@param servers ServerState[] ---@param props ServerGroupProps local function UninstalledServers(servers, props) - return Ui.Node(Data.list_map(function(server) + return Ui.Node(Data.list_map(function(_server) + ---@type ServerState + local server = _server local is_prioritized = props.prioritized_servers[server.name] local is_expanded = props.expanded_server == server.name return Ui.Node { @@ -317,9 +320,10 @@ local function UninstalledServers(servers, props) settings.current.ui.icons.server_uninstalled, is_prioritized and "LspInstallerHighlighted" or "LspInstallerMuted", }, - { " " .. server.name, "LspInstallerMuted" }, - Data.when(server.uninstaller.has_run, { " (uninstalled)", "Comment" }), - Data.when(server.deprecated, { " deprecated", "LspInstallerOrange" }) + { " " .. server.name .. " ", "LspInstallerMuted" }, + { server.hints, "Comment" }, + Data.when(server.uninstaller.has_run, { " (uninstalled) ", "Comment" }), + Data.when(server.deprecated, { "deprecated ", "LspInstallerOrange" }) ), }, Ui.Keybind(settings.current.ui.keymaps.toggle_server_expand, "EXPAND_SERVER", { server.name }), @@ -361,7 +365,8 @@ end ---@param servers table ---@param expanded_server string|nil ---@param prioritized_servers string[] -local function Servers(servers, expanded_server, prioritized_servers) +---@param server_name_order string[] +local function Servers(servers, expanded_server, prioritized_servers, server_name_order) local grouped_servers = { installed = {}, queued = {}, @@ -375,7 +380,8 @@ local function Servers(servers, expanded_server, prioritized_servers) } -- giggity - for _, server in pairs(servers) do + for _, server_name in ipairs(server_name_order) do + local server = servers[server_name] if server.installer.is_running then grouped_servers.installing[#grouped_servers.installing + 1] = server elseif server.installer.is_queued then @@ -443,6 +449,7 @@ local function create_initial_server_state(server) name = server.name, is_installed = server:is_installed(), deprecated = server.deprecated, + hints = tostring(ServerHints.new(server)), metadata = { homepage = server.homepage, install_timestamp_seconds = nil, -- lazy @@ -491,7 +498,12 @@ local function init(all_servers) return Help(state.is_current_settings_expanded, state.vader_saber_ticks) end), Ui.When(not state.is_showing_help, function() - return Servers(state.servers, state.expanded_server, state.prioritized_servers) + return Servers( + state.servers, + state.expanded_server, + state.prioritized_servers, + state.server_name_order + ) end), } end @@ -499,14 +511,20 @@ local function init(all_servers) ---@type table local servers = {} + ---@type string[] + local server_name_order = {} for i = 1, #all_servers do local server = all_servers[i] servers[server.name] = create_initial_server_state(server) + server_name_order[#server_name_order + 1] = server.name end + table.sort(server_name_order) + ---@class StatusWinState ---@field prioritized_servers string[] local initial_state = { + server_name_order = server_name_order, servers = servers, is_showing_help = false, is_current_settings_expanded = false, diff --git a/lua/nvim-lsp-installer/ui/status-win/server_hints.lua b/lua/nvim-lsp-installer/ui/status-win/server_hints.lua new file mode 100644 index 00000000..daf5e9b6 --- /dev/null +++ b/lua/nvim-lsp-installer/ui/status-win/server_hints.lua @@ -0,0 +1,41 @@ +---@class ServerHints +---@field server Server +local ServerHints = {} +ServerHints.__index = ServerHints + +function ServerHints.new(server) + return setmetatable({ server = server }, ServerHints) +end + +---@param language string +---@return boolean +function ServerHints:is_language_equal(language) + local match_start, match_end = self.server.name:find(language, 1, true) + -- This is somewhat... arbitrary + return match_start ~= nil + and (match_end - match_start) >= 2 -- the match need to be at least 2 in length + -- match needs to start in the beginning - if it's not, then the total string lengths cannot differ too much + and (match_start == 1 or (match_start < 3 and (math.abs(#self.server.name - #language) < 4))) +end + +function ServerHints:get_hints() + local hints = {} + if self.server.languages then + for _, language in ipairs(self.server.languages) do + if not self:is_language_equal(language) then + hints[#hints + 1] = language + end + end + end + return hints +end + +function ServerHints:__tostring() + local hints = self:get_hints() + if #hints == 0 then + return "" + end + return "(" .. table.concat(hints, ", ") .. ")" +end + +return ServerHints -- cgit v1.2.3-70-g09d2