fix: Marks on current row should only be hidden for current buffer

## Details

Currently when LSP signature help is opened the first line virtual
text is hidden. This is because the cursor for that buffer is on
the first line, however this should not matter since that buffer is
not active.

To fix this check whether the buffer being rendered is the current
buffer when getting the cursor position. If it is not the current
buffer use nil as the row value and render marks when row is nil.

Refactor the logic for showing vs. hiding a mark into a local function
rather than a large if statement. This lets us comment what's going
on and I have a feeling that logic will need to evolve in other ways
over time.

Add a todo doc to keep track of various ideas that I haven't fully
ruled out implementing.

Add plugin configuration and checkboxes of confirmations as
fields to bug report template.

Author: MeanderingProgrammer

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -35,9 +35,22 @@ body:
       required: true
   - type: textarea
     attributes:
-      label: Healthcheck output (:checkhealth render-markdown)
+      label: Healthcheck output
+      description: Output of :checkhealth render-markdown
     validations:
       required: true
+  - type: textarea
+    attributes:
+      label: Plugin configuration
+      description: Your configuration for this plugin.
+    validations:
+      required: true
+  - type: checkboxes
+    attributes:
+      label: Confirmations
+      options:
+      - label: I have provided markdown text for any screenshots used in my description & understand that my issue will be closed if I have not
+        required: true
   - type: textarea
     attributes:
       label: Additional information

doc/todo.md

@@ -0,0 +1,6 @@
+# TODOs
+
+- See if there is a stable way to align table cells according to delimiter
+  alignment info.
+- Add a performance test suite to measure improvements, relative value on
+  same hardware should still be useful.

lua/render-markdown/ui.lua

@@ -78,10 +78,30 @@ M.refresh = function(buf, mode, parse)
         cache.marks[buf] = marks
     end
 
-    -- Render marks based on anti-conceal behavior and current row
-    local row = vim.api.nvim_win_get_cursor(util.buf_to_win(buf))[1] - 1
+    ---Render marks based on anti-conceal behavior and current row
+    ---@param mark render.md.Mark
+    ---@param row? integer
+    ---@return boolean
+    local function should_show_mark(mark, row)
+        -- Anti-conceal is not enabled -> all marks should be shown
+        if not state.config.anti_conceal.enabled then
+            return true
+        end
+        -- Row is not known means buffer is not active -> all marks should be shown
+        if row == nil then
+            return true
+        end
+        -- Mark is not concealable -> mark should always be shown
+        if not mark.conceal then
+            return true
+        end
+        -- Show mark if it is not on the current row
+        return mark.start_row ~= row
+    end
+
+    local row = util.cursor_row(buf)
     for _, mark in ipairs(marks) do
-        if not state.config.anti_conceal.enabled or not mark.conceal or mark.start_row ~= row then
+        if should_show_mark(mark, row) then
             -- Only ensure strictness if the buffer was parsed this request
             -- The order of events can cause our cache to be stale
             mark.opts.strict = parse

lua/render-markdown/util.lua

@@ -18,6 +18,15 @@ M.buf_to_win = function(buf)
     return vim.fn.bufwinid(buf)
 end
 
+---@param buf integer
+---@return integer?
+M.cursor_row = function(buf)
+    if vim.api.nvim_get_current_buf() ~= buf then
+        return nil
+    end
+    return vim.api.nvim_win_get_cursor(M.buf_to_win(buf))[1] - 1
+end
+
 ---@param buf integer
 ---@return integer
 M.get_width = function(buf)