open_html doesn't work if getOption("browser") is a function

[idavydov]

I noticed that if I set options(browser=function(url) {...}) then open_html setting is ignored.

I think it's a valid option and is correctly used e.g. by browserURL().

Would it maybe be possible to allow browser which is a function? I am using this to view html's from a remote host using a combination of ssh port forwarding, servr and opener.

Not sure if my solution is optimal, by happy to document it in a wiki.

[jalvesaq]

When the browser option isn't a string, R.nvim receives an empty string as the browser's name, and, then, it calls either open (Windows or macOS) or xdg-open (Linux) to open the HTML.

[jalvesaq]

Not sure if my solution is optimal, by happy to document it in a wiki.

Yes, please.

Perhaps, if the option "browser" is a function R.nvim should use R to open the HTML document. Or, maybe, it should always use R to open HTML documents.

[idavydov]

Here the first draft; at the end I mention the problem I currently have.

Yes, for the remote setup it would be great if R.nvim could call browseURL() via R. I do not see any downsides of this approach.

[jalvesaq]

Here the first draft; at the end I mention the problem I currently have.

Thanks!

Could you try the branch "browseURL", please?

[idavydov]

Great, thanks a lot.

browserURL seem to work great with .Rmd. Quarto render doesn't open preview automatically, as far as I understand. For quarto preview one could set:

        quarto_preview_args = ", port=.free_port()",

I will update the wiki once this is in main.

So all seem to work great.

[jalvesaq]

Merged now. Thanks for reporting the issue and for writing the page on the Wiki!

[idavydov]

sorry for reopening this discussion. I am now getting suspicious messages after rendering an R Markdown file.

^I[C]: in function 'jobstart'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:128: in function 'start'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/doc.lua:223: in function 'load_html'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/doc.lua:237: in function 'open'
^I[string ":lua"]:1: in main chunk
^I[C]: in function 'execute'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:48: in function 'exec_stdout_cmd'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:69: in function <<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:58>
stack traceback:
^I[C]: in function 'execute'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:48: in function 'exec_stdout_cmd'
^I<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:69: in function <<...>/.local/share/nvim/lazy/R.nvim/lua/r/job.lua:58>

[jalvesaq]

This isn't easy to debug without replicating locally. It's necessary to put nvimcom.verbose=6 and/or enable logging in rnvimserver to know what message is causing the problem.

[idavydov]

So let's say I have a minimal .Rmd:

```{r}
cat("hi")

Every time I press `<localleader>kh` I get this output with `nvimcom.verbose=6`:

nvim.interlace.rmd("a.Rmd", outform = "html_document", rmddir = "", envir = .GlobalEnv)

processing file: a.Rmd

output file: a.knit.md

<...>/bin/pandoc +RTS -K512m -RTS a.knit.md --to html4 --from markdown+au tolink_bare_uris+tex_math_single_backslash --output a.html --lua-filter /usr/l ocal/lib/R/site-library/rmarkdown/rmarkdown/lua/pagebreak.lua --lua-filter /us r/local/lib/R/site-library/rmarkdown/rmarkdown/lua/latex-div.lua --embed-resou rces --standalone --variable bs3=TRUE --section-divs --template /usr/local/lib /R/site-library/rmarkdown/rmd/h/default.html --no-highlight --variable highlig htjs=1 --variable theme=bootstrap --mathjax --variable 'mathjax-url=https://ma thjax.rstudio.com/latest/MathJax.js?config=TeX-AMS-MML_HTMLorMML' --include-in -header /tmp/Rtmp1xCtyo/rmarkdown-str1f8fb64200f6f.html [WARNING] This document format requires a nonempty