-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Raise default values #112
Comments
No, 5000px is not a sane default. I agree 768 and 1024 is possibly a bit low nowadays, but the purpose of this extension is to work out-of-the-box. That's unfortunate you didn't pay attention, sorry for you. |
I'm not blaming you for it, of course! It's our bad. I'm just thinking about protecting others from the same dumb mistake. wdy think is a sane default size? I used to put 3000 in the last years, but in this specific project we use the photos for print as well, that's why I went up to 5000. |
We've had serious issues due to installing image_autoresize without thinking about changing the default values.
So the resize defaults
768px
and1024px
kicked in, and we just discovered this after hundreds of images had been uploaded by FE users and were saved in a small format.We've fixed this for us for the future, but I think it might make sense to raise these default values eg. to 5000px.
Making smaller is always possible, but larger not.
The text was updated successfully, but these errors were encountered: