-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Remove regexp from PreviewManager list #40990
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
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure about the TTF split.
I've added the TTF split, based on the assumption that regexp syntax in this list is to be deprecated. Unsure about that too :-) |
I think the mediaviewer should interpret the regexp in the mimetype, but that would be more complex. For now, the quick fix is to split the HEIC and HEIF formats.
|
I have reverted the change about TTF and just left the split about HEIC and HEIF, based on what JP commented. Changelog has been added as well. |
7e904af
to
0641eaa
Compare
Kudos, SonarCloud Quality Gate passed! |
We might need to open a ticket for the TTF fonts preview at some point, although I don't know if there is anyone interested in that. |
Suggested fix for owncloud/files_mediaviewer#914
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: