How to give custom marks higher priority?

[chrisgrieser]

I have created this simple custom mark that solely displays the current line. However, I am not sure what I have to do to make the "current line" mark override the other marks, e.g. from gitsign.

require("scrollbar.handlers").register("currentLine", function(bufnr)
	local currentLnum = fn.line(".")
	return {{ line = currentLnum, text = "", type = "Hint", level = 0}}
end)

I tried level = 6 and level = 0 (assuming those levels correspond to priority from the config), but with both of those, the gitsign marks take priority over my custom mark.

[petertriho]

Sorry, I really need to improve the docs on this. Level is for "text" index used under config.marks.[type].text
To set a higher priority, you are using "Hint" there, so in the setup:

require("scrollbar").setup({
  marks = {
    Hint = {
      priority = 0,
    }
  }
})

[petertriho]

Ah sorry, it is a lower number (so 0), but I suspect it's because the cursor mark is only being rendered when something else (like lsp) is rendering the scrollbar that the mark shows up. You should try and use autocmd events to render the cursor

[petertriho]

I really should just add this to the included handlers

Example:

        require("scrollbar").setup({
            marks = {
                Cursor = { text = "", priority = -1, highlight = "Normal" },
            },
        })

        vim.api.nvim_create_autocmd({ "CursorHold", "CursorHoldI", "CursorMoved", "CursorMovedI" }, {
            callback = function()
                local current_line = vim.fn.line(".") - 1
                local bufnr = vim.fn.bufnr("%")

                local utils = require("scrollbar.utils")
                local scrollbar_marks = utils.get_scrollbar_marks(bufnr)
                scrollbar_marks.cursor = { { line = current_line, text = "", type = "Cursor" } }
                utils.set_scrollbar_marks(bufnr, scrollbar_marks)

                require("scrollbar").render()
            end
        })

[petertriho]

Priority should be fixed now 819de9b

[petertriho]

Cursor handler now included

[chrisgrieser]

Fantastic, thank you! Works as expected