wfxr / tmux-fzf-url

🚀 Quickly open urls on your terminal screen!
Other
526 stars 42 forks source link

Strip escape sequences before fzf selection #44

Open pmodin opened 8 months ago

pmodin commented 8 months ago

This patch strips fzf items with ansifilter1 if it's available, also mentioning it in the README.

I'm not sure why db053fa17ec51aae3be80a99ad24e8c30f620770 adds them, neither tmux-fpp nor tmux-urlview seems to do this. However, I did end up with these sequences among the selection items (and the browser tried to open them, with 404s as results), so I feel a strip here is needed. Unless a revert of db053fa17ec51aae3be80a99ad24e8c30f620770 is better, but I don't understand the reasoning behind it so strip it is.

Thanks for this plugin.