smart-open.nvim icon indicating copy to clipboard operation
smart-open.nvim copied to clipboard

fix issues/70

Open tim-tim707 opened this issue 1 year ago • 1 comments

Changes are done without understanding most of this plugin. I found a hack to make it work but I'm sure there is a better way to do it that I don't know of.

Explanations of the bug can be found on my message here: https://github.com/danielfalk/smart-open.nvim/issues/70#issuecomment-2582802814

Maybe the flag on ripgrep is enough and the vim.fs.normalize are only here because my sqlite3 is infected with bad file path ? Right now, it doesn't work if I don't keep the vim.fs.normalize. Some directories are half normalized and half not, which may come from a data provider I am not aware of. I haven't fully tested properly my changes, maybe there are data paths that do not go through a normalize call and in that case it could fail like previously

tim-tim707 avatar Jan 10 '25 15:01 tim-tim707

I don't know how the default smart-open is supposed to work so I need to test this on linux, but I think this fix breaks the "current directory listing". I opened a new folder for another project but none of the files would show up. require("telescope.builtin").find_files() works fine for this directory though, so I think it is a problem with my changes

tim-tim707 avatar Jan 13 '25 09:01 tim-tim707

This should be fixed now. Thanks for your PR. Feel free to reopen if it doesn't work after update

danielfalk avatar Nov 16 '25 22:11 danielfalk