R-nvim / R.nvim

Neovim plugin to edit R files
GNU General Public License v3.0
155 stars 16 forks source link

Remove support for Evince, Okular and Qpdfview #32

Closed jalvesaq closed 5 months ago

jalvesaq commented 7 months ago

I can't see any reason to keep support for Evince, Okular, and qpdfview. They are better than Zathura for annotation, filling forms, etc. But we only need to display the newly generated PDF quickly, and Zathura excels in this task.

So, I propose to delete the files evince.lua, okular.lua, and qpdfview.lua. It will still be possible to use them as "generic" pdf viewers, but without the SyncTeX support.

@PMassicotte ? @she3o ?

jalvesaq commented 7 months ago

I can also keep the Lua files but don't use them. So, if someone wanted to implement the SyncTeX support, it would be easier.

PMassicotte commented 7 months ago

I will talk for myself, but I only used Zathura. So it is all good with me :)

jalvesaq commented 7 months ago

I will disable the other three, but keep their Lua files for at least a couple of weeks after the inauguration of R.nvim. This way, if anybody wants to support them in the future, it will be possible to recover the Lua code.