fix_lsp_window shrinks LSP hover float: nvim_win_text_height already accounts for conceal_lines

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#720 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 21 jours

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
78/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
lua, markdown, neovim
Domaine
devtools

Piste de recherche

Commencez par lua/render-markdown/lib/compat.lua et examinez M.fix_lsp_window, en particulier la façon dont il utilise nvim_win_text_height() et les extmarks conceal_lines. Vérifiez le moment du rendu et le comportement de la hauteur de la fenêtre décrits dans le rapport ; la reproduction suggérée utilise Neovim avec sqls affichant au survol une table large. Le travail est terminé lorsque la fenêtre flottante du survol affiche la table entièrement rendue à la hauteur mesurée, sans être trop réduite.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Describe the bug

LSP hover floats get shrunk to a few lines when the hover document contains a rendered markdown table, because compat.fix_lsp_window() subtracts one line per conceal_lines extmark from nvim_win_text_height(), which already returns the post-fold height.

With sqls hovering a 14-column table (18-line hover doc) the float ends up 112x3 instead of 112x19; the rendered table is unreadable (only the first rows visible) even though it was rendered correctly.

:RenderMarkdown disable gives the normal 112x18 float, so this is the plugin, not lspconfig/sqls.

To reproduce

Environment: NVIM v0.13.0-dev-2042+g74f3ce7970, render-markdown 8e87953 (2026-10-10), default opts (file_types = { 'markdown', 'codecompanion' }).

  1. sqlite db with e.g. CREATE TABLE big(a INT, b TEXT, c REAL, d BLOB, e INT, f TEXT, g INT, h TEXT, i INT, j TEXT, k INT, l TEXT, m INT, n TEXT);
  2. sqls connected, cursor on the table name, :lua vim.lsp.buf.hover()

The hover doc is:

# `big` table

| Name   | Type   | Primary key … | … |
| :--------------- | :--------------- | :---------------------- | … |
| `a` | `INT` | `NO` | `-` |  |
…
Actual
win 112x3   nvim_win_text_height = 18 at the time fix_lsp_window runs, 19 after the render settles
extmarks: 15 with conceal_lines, 17 rows of virt_lines (the drawn table)
height = 18 - 15 = 3
Expected

Float large enough for the rendered content (112x19).

Root cause

lua/render-markdown/lib/compat.lua:

-- 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 then
        height = height - 1
    end
end
if height < vim.api.nvim_win_get_height(win) then
    vim.api.nvim_win_set_height(win, height)
end

Verified on this Neovim build that nvim_win_text_height() already excludes lines collapsed by conceal_lines marks: for the same 18-line hover doc, a normal (non-float) markdown buffer after render shows the table on rows 3..20 (18 rows, checked with a pty + screenstring()), while nvim_win_text_height() reports 19 (18 rows + the trailing empty line). So the subtraction is a second deduction and the more conceal_lines marks a table produces, the smaller the float gets.

Note also that the function only ever shrinks (if height < ...), so nothing restores the height when the measurement is too small.

Suggested fix

Trust the measurement, and re-measure after the marks have settled:

function M.fix_lsp_window(buf, win, extmarks)
    local has_lsp = pcall(vim.api.nvim_win_get_var, win, 'lsp_floating_bufnr')
    if not has_lsp then
        return
    end
    local env = require('render-markdown.lib.env')
    local str = require('render-markdown.lib.str')
    -- width handling unchanged
    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
    vim.schedule(function()
        if not vim.api.nvim_win_is_valid(win) then
            return
        end
        local height = vim.api.nvim_win_text_height(win, {}).all
        if height ~= vim.api.nvim_win_get_height(win) then
            pcall(vim.api.nvim_win_set_height, win, height)
        end
    end)
end

With this patch the same hover shows 112x19 with the whole table visible. Small terminals stay fine: nvim_win_set_height is clamped by Neovim to the available rows (16-line terminal -> height 14, scrolling inside the float still works).

Checklist
  • I have read the CONTRIBUTING guide
  • I have searched existing issues
Langage dominant
Lua
Étoiles
5.2k
Forks
155
Merge moyen
5 j 10 h
PR mergées (30 j)
1

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de MeanderingProgrammer/render-markdown.nvim

Toutes les issues de MeanderingProgrammer/render-markdown.nvim

Issues similaires

Plus d'issues Lua

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.