Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Why no webp? #109

Open
xeruf opened this issue Apr 13, 2024 · 1 comment
Open

Why no webp? #109

xeruf opened this issue Apr 13, 2024 · 1 comment
Assignees
Labels
enhancement helpwanted quickfix small change necessary to fix this

Comments

@xeruf
Copy link
Contributor

xeruf commented Apr 13, 2024

Just tried to embed a webp image, but the parser was not taking it because it is not in the matching expression:
https://github.com/novoid/lazyblorg/blob/master/lib/orgparser.py#L131

    CUST_LINK_IMAGE_REGEX = re.compile(r'^\[\[tsfile:([^\]]+\.(png|jpg|jpeg|svg|gif))+(\]\[(.+))?\]\]$')

Simply forgotten or intentional?

@novoid
Copy link
Owner

novoid commented Apr 15, 2024

I had no idea what webp is and I still don't know how its support by imagemagick is.

When you find out that this ought not to be an issue, I can extend the regex.

@novoid novoid self-assigned this Apr 15, 2024
@novoid novoid added enhancement helpwanted quickfix small change necessary to fix this labels Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement helpwanted quickfix small change necessary to fix this
Projects
None yet
Development

No branches or pull requests

2 participants