fix: disable line wrapping in LSP hover docs in some cases

## Details

Issue: #408

For narrow windows adding the language icon above code blocks can lead
to text wrapping which will lead to an extra line under the code block.

There are really only 2 ways to fix this:

1. Increase the window width such that it is wide enough to avoid wrapping.
2. Disable line wrapping.

In the first case we end up needing to accurately calculate the width we
need to avoid the line wrapping and also check that the new width does
not cause the window to flow out of bounds. There's lots of edge cases
to these and the end result doesn't look particularly great since it'll
add extra space off the side of these narrow doc buffers.

So instead we take the much simpler approach of disabling line wrapping
on these buffers, however we do this only when the content in the buffer
does not need line wrapping, i.e. the width of the longest line is <=
the width of the window. This should work for most situations but I'm
sure there will be some edge cases to it.

Author: MeanderingProgrammer

lua/render-markdown/core/ui.lua

@@ -158,7 +158,7 @@ function M.run_update(buf, win, change)
         local hidden = config:hidden(mode, row)
         local extmarks = buffer:get_marks()
         if initial then
-            Compat.lsp_window_height(win, extmarks)
+            Compat.lsp_window_height(buf, win, extmarks)
             state.on.initial({ buf = buf, win = win })
         end
         for _, extmark in ipairs(extmarks) do

lua/render-markdown/health.lua

@@ -5,7 +5,7 @@ local state = require('render-markdown.state')
 local M = {}
 
 ---@private
-M.version = '8.3.6'
+M.version = '8.3.7'
 
 function M.check()
     M.start('version')

lua/render-markdown/lib/compat.lua

@@ -5,29 +5,40 @@ M.uv = vim.uv or vim.loop
 M.has_10 = vim.fn.has('nvim-0.10') == 1
 M.has_11 = vim.fn.has('nvim-0.11') == 1
 
+---@param buf integer
 ---@param win integer
 ---@param extmarks render.md.Extmark[]
 ---@see vim.lsp.util.open_floating_preview
-function M.lsp_window_height(win, extmarks)
-    -- This is a fragile way of identifying whether this is a floating LSP buffer,
-    -- comes from the implementation and not from any documentation
-    local lsp_buf = pcall(vim.api.nvim_win_get_var, win, 'lsp_floating_bufnr')
-    if not lsp_buf then
+function M.lsp_window_height(buf, win, extmarks)
+    -- this is a fragile way of identifying whether this is a floating LSP
+    -- window, comes from the implementation and not from any documentation
+    local has_lsp = pcall(vim.api.nvim_win_get_var, win, 'lsp_floating_bufnr')
+    if not has_lsp then
         return
     end
-    local concealed = 0
+
+    local Env = require('render-markdown.lib.env')
+    local Str = require('render-markdown.lib.str')
+
+    -- account for conceal lines marks allowing us to reduce window height
+    local height = vim.api.nvim_win_text_height(win, {}).all
     for _, extmark in ipairs(extmarks) do
         if extmark:get().opts.conceal_lines ~= nil then
-            concealed = concealed + 1
+            height = height - 1
         end
     end
-    if concealed == 0 then
-        return
-    end
-    local height = vim.api.nvim_win_text_height(win, {}).all - concealed
     if height < vim.api.nvim_win_get_height(win) then
         vim.api.nvim_win_set_height(win, height)
     end
+
+    -- disable line wrapping if it is not needed to contain the text
+    local width = 0
+    for _, line in ipairs(vim.api.nvim_buf_get_lines(buf, 0, -1, false)) do
+        width = math.max(width, Str.width(line))
+    end
+    if width <= vim.api.nvim_win_get_width(win) then
+        Env.win.set(win, 'wrap', false)
+    end
 end
 
 ---@param cause string