Use full mode when deciding to render or not

resolves: #43

# Details

Currently we truncate mode to the first character when deciding to
render or not. This reduces the amount of control we have, as described
in the issue above, where not rendering during operator pending mode
`no` would improve navigation.

The fix is to remove any truncation logic so we access the full mode.

This on its own handles operator pending correctly as `no` mode is
no longer considered a render mode. So the defaults do not change.

There could be some edge cases that crop up with different mode
combinations so may need to update the default set of rendered modes,
but seems to work as expected from some basic use.

Author: MeanderingProgrammer

lua/render-markdown/init.lua

@@ -247,8 +247,8 @@ function M.setup(opts)
         group = group,
         callback = function(event)
             local render_modes = state.config.render_modes
-            local prev_rendered = vim.tbl_contains(render_modes, string.sub(vim.v.event.old_mode, 1, 1))
-            local should_render = vim.tbl_contains(render_modes, string.sub(vim.v.event.new_mode, 1, 1))
+            local prev_rendered = vim.tbl_contains(render_modes, vim.v.event.old_mode)
+            local should_render = vim.tbl_contains(render_modes, vim.v.event.new_mode)
             -- Only need to re-render if render state is changing. I.e. going from normal mode to
             -- command mode with the default config, both are rendered, so no point re-rendering
             if prev_rendered ~= should_render then

lua/render-markdown/ui.lua

@@ -16,7 +16,7 @@ M.namespace = vim.api.nvim_create_namespace('render-markdown.nvim')
 
 ---@param buf integer
 M.schedule_refresh = function(buf)
-    local mode = vim.fn.mode()
+    local mode = vim.fn.mode(true)
     vim.schedule(function()
         M.refresh(buf, mode)
     end)