A nice #Laravel zero downtime deployment script 🤓
gist.github.com/Sairahcaz/104019bf733663668610fdd18590c509
•
Back
Yes :D, it was to high. Max here is 4000x4000 px, on twitter/linkedin it was working though...
•
I think we need to think more on it as we are already compressing images after upload. So we need to allow if it fall in accept ratio.
cc: @CamKem (any thoughts?)
•
@Sairahcaz2k was is the aspect ratio that failed or the pixel size?
4000 x 4000 is a very huge photo, if I screenshot my entire screen it' only 3456 × 2234, this is what we've based it off. Even the ultrawide iphone 15 Pro/Pro Max camera only takes a photo 3024 × 4032. So I assume maybe you used Devtools to capture the entire page even outside of the viewport?
@MrPunyapal - I guess we could consider allowing larger, doubling it would be ok, especially now that we restrict the aspect ratio, which is really whats most important for viewability. Either way, you might find if you took an 8000 x 8000 pixel photo of text, when it's compressed to 1000 x 1000 the text might not be readable, we'd have to look into it more.
•
In response to @CamKem
Btw that even happens with me on Twitter when I upload really high quality code snippet it just get bad on readability.