Add custom media_extensions support

iturdikulov commented 7 months ago

Customize insert image and preview image files list.

Proposed change

Type of change

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (thank you!)
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation update

Additional information

This is useful to add optional filetypes into filtered list (for example telescope-media-files optionally supporting svg preview). With this solution user can control which media files he want to see in telescope windows.

This PR fixes or closes issue: fixes #301
This PR is related to issue: #301

Checklist

[x] I am running the latest version of the plugin.
[x] The code change is tested and works locally (preview, media preview, insert link).
[x] There is no commented out code in this PR.
[x] The code has been formatted using Stylua (a .stylua.toml file is provided)
[ ] The code has been checked with luacheck (a .luacheckrc file is provided)
[ ] The README.md has been updated according to this change. No configuration section?
[x] The doc/telekasten.txt helpfile has been updated according to this change.

lambtho12 commented 7 months ago

Is there a reason to expose this as an option? This seems a bit unnecessary and cumbersome. Why not just expanding the hard-coded list to add the most common image extensions?

iturdikulov commented 7 months ago

Yes, main here few reasons (at least for me):

Telekasten browser_media is used to generate media files list and for example my preview plugin is not supporting svg files and I just don't want to see them here (lets assume I have a lot of svg files). Or for example I want to preview some rare file-type like .tiff, but it's not in a supported files list. Here a lot of combinations.

I think chafa (which used by telescope-media-files plugin) support a lot file-types (probably anything which imagemagic support, magick identify -list format gives 302 file formats) and managing this file-types from configuration can be useful.

lambtho12 commented 7 months ago

Thanks for explaining your implementation. I am fine with having it as an option then. Just address the review so the doc matches the default config and I will merge.

lambtho12 commented 7 months ago

Thank you ! Feel free to tackle any other opened issue :smile: We definitely need more contributors.

nvim-telekasten / telekasten.nvim