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

J5.0.3 Media Manager doesn't like capital names like file.JPG #43315

Closed
thetexian opened this issue Apr 19, 2024 · 7 comments
Closed

J5.0.3 Media Manager doesn't like capital names like file.JPG #43315

thetexian opened this issue Apr 19, 2024 · 7 comments

Comments

@thetexian
Copy link

Steps to reproduce the issue

File named IMG_455.JPG
In media manager edit mode, resized and cropped. Upon save, nothing changed. No messages, no acknowledgement.

Expected result

Expected result is change to image

Actual result

No change. No message

System information (as much as possible)

J 5.0.3 Media Manager

Additional comments

When file renamed to IMG_455.jpg, worked as designed
Media manager doesn't like capitalized filetypes apparently.

@brianteeman
Copy link
Contributor

I just added this for you to the relevant open issue. #43220

@dgrammatiko
Copy link
Contributor

Should be fixed with #43295

@richard67
Copy link
Member

richard67 commented Apr 20, 2024

Closing as fixed with #43295 . Please test.

@brianteeman
Copy link
Contributor

sorry to say but 43295 does not fix this

@richard67 richard67 reopened this Apr 20, 2024
@dgrammatiko
Copy link
Contributor

FWIW it's not the filename the problem but the capitalized extension. I thought the mentioned PR would fix this but I had to revert some code when I realized that the canvas element has some inconsistent functionality across browsers and also realizing that introducing more formats for the editing part is a major change that can't happen before 6.0 or it would break all existing plugins.

@dgrammatiko
Copy link
Contributor

Please test: #43325

@richard67
Copy link
Member

Closing as having a pull request. Please test #43325 . Thanks in advance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants