SI
SI
discoversearch

We've detected that you're using an ad content blocking browser plug-in or feature. Ads provide a critical source of revenue to the continued operation of Silicon Investor.  We ask that you disable ad blocking while on Silicon Investor in the best interests of our community.  If you are not using an ad blocker but are still receiving this message, make sure your browser's tracking protection is set to the 'standard' level.
SI - Site Forums : Questions and Answers with SI Admin (s)
 Public ReplyPrvt ReplyMark as Last ReadFilePrevious 10Next 10PreviousNext  
To: Sun Tzu who wrote (4746)9/20/2023 6:53:48 PM
From: SI Dmitry (code monkey)6 Recommendations

Recommended By
Bill
Eric L
longz
Roads End
Sun Tzu

and 1 more member

   of 4890
 
Dear Sun Tzu:

I have duplicated your issue.

I looked at our source code. If the image is more than 1367 pixels in height or width, the code resizes the image to a maximum height or width of 960 pixels.

This is some really old PHP code (which also generates an incredibly small thumbnail). All of this code needs to be replaced, preferably eventually by something other than PHP. The height & width restrictions are very outdated; my monitor is 2560x1440. The limitation should really be based upon file size, not image dimensions, especially considering that image compression has come a long way, and that our site's javascript code automatically resizes the image to fit the browser window now (a change I made).

I will look to make an update to address this soon. Thanks for your patience and pointing this out.

Regards,

@Dima
Report TOU ViolationShare This Post
 Public ReplyPrvt ReplyMark as Last ReadFilePrevious 10Next 10PreviousNext  

HomeMailHotSubjectMarksPeopleMarksKeepersSettings
Terms Of UseContact UsCopyright/IP PolicyPrivacy PolicyAbout UsFAQAdvertise on SI
© 2025 Knight Sac Media.  Data provided by Twelve Data, Alpha Vantage, and CityFALCON News