Skip to content

Add jp2 support - #2587

Closed
mexthecat wants to merge 1 commit into
nextcloud:masterfrom
mexthecat:master
Closed

mexthecat wants to merge 1 commit into
nextcloud:masterfrom
mexthecat:master

Conversation

@mexthecat

Copy link
Copy Markdown

Could you please add jpeg2000 to the previewSupportedMimes.
Would be cool.
Thanx

@github-actions

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@skjnldsv

Copy link
Copy Markdown
Member

Thanks for this, and sorry it sat so long.

Your change was right, it just had nothing behind it. This list is filtered
against the server's enabled preview providers, and nothing in Nextcloud
claimed image/jp2, so the entry got dropped again on every server. That is
presumably why it never went anywhere.

So we did the missing half. nextcloud/server#64603 adds OC\Preview\JP2,
which is merged and lands in 36, and the entry itself is now in the viewer
library as nextcloud-libraries/nextcloud-viewer#53 with your co-authorship on
the commit.

Closing here because this repository is frozen: the viewer moved to
https://github.com/nextcloud-libraries/nextcloud-viewer as @nextcloud/viewer
and master no longer takes changes, only the stable branches until they go EOL.

One caveat so you know what to expect. ImageMagick is the only backend that
reads JPEG 2000 (libgd has no support at all, and Imaginary returns 406), so
like TIFF and HEIC the provider is off until an admin adds OC\Preview\JP2 to
enabledPreviewProviders. It will not work out of the box on most servers.

@skjnldsv skjnldsv closed this Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants