doubledee Posted June 8, 2012 Share Posted June 8, 2012 What do you think are reasonable File-Size restrictions for a form which allows users to upload their Photo? Right now, my code allows between 4kb and 100kb. Debbie Quote Link to comment Share on other sites More sharing options...
Philip Posted June 8, 2012 Share Posted June 8, 2012 Depends on the size of the quality you're wanting, and the dimensions. Quote Link to comment Share on other sites More sharing options...
scootstah Posted June 8, 2012 Share Posted June 8, 2012 I don't think a minimum size makes any sense, especially not 4KB. Your profile picture is only 2.57KB. I think somewhere in the neighborhood of 100-150KB for 90X90px images should be acceptable. Quote Link to comment Share on other sites More sharing options...
doubledee Posted June 8, 2012 Author Share Posted June 8, 2012 I don't think a minimum size makes any sense, especially not 4KB. Your profile picture is only 2.57KB. It does, because it crashes one of the functions in my script. I think its a GD issue. Can't remember which one, but it is a know issue, so I added in that code last January. (I'm going back and trying to do a better job commenting this script for just this reason!!!!) I think somewhere in the neighborhood of 100-150KB for 90X90px images should be acceptable. @King Philip: These are just for Profile Pics. @Scootstah: So maybe I'll bump it up to 150kb. Debbie Quote Link to comment Share on other sites More sharing options...
scootstah Posted June 8, 2012 Share Posted June 8, 2012 I don't think a minimum size makes any sense, especially not 4KB. Your profile picture is only 2.57KB. It does, because it crashes one of the functions in my script. I think its a GD issue. Can't remember which one, but it is a know issue, so I added in that code last January. That's not how you fix bugs. And 4KB is still way too high. Quote Link to comment Share on other sites More sharing options...
doubledee Posted June 9, 2012 Author Share Posted June 9, 2012 I don't think a minimum size makes any sense, especially not 4KB. Your profile picture is only 2.57KB. It does, because it crashes one of the functions in my script. I think its a GD issue. Can't remember which one, but it is a know issue, so I added in that code last January. That's not how you fix bugs. And how would you fix it? And 4KB is still way too high. Too high for what? Debbie Quote Link to comment Share on other sites More sharing options...
scootstah Posted June 9, 2012 Share Posted June 9, 2012 I don't think a minimum size makes any sense, especially not 4KB. Your profile picture is only 2.57KB. It does, because it crashes one of the functions in my script. I think its a GD issue. Can't remember which one, but it is a know issue, so I added in that code last January. That's not how you fix bugs. And how would you fix it? I would figure out what the problem is and then find a solution. And 4KB is still way too high. Too high for what? The minimum limit is too high. It is not unreasonable for a small profile picture/thumbnail to be <4KB. Your profile picture is under 3KB. Quote Link to comment Share on other sites More sharing options...
doubledee Posted June 9, 2012 Author Share Posted June 9, 2012 I don't think a minimum size makes any sense, especially not 4KB. Your profile picture is only 2.57KB. It does, because it crashes one of the functions in my script. I think its a GD issue. Can't remember which one, but it is a know issue, so I added in that code last January. That's not how you fix bugs. And how would you fix it? I would figure out what the problem is and then find a solution. Great, then you can help us over here in this thread... http://www.phpfreaks.com/forums/index.php?topic=360669.new;topicseen#new And 4KB is still way too high. Too high for what? The minimum limit is too high. It is not unreasonable for a small profile picture/thumbnail to be <4KB. Your profile picture is under 3KB. Well, see the link above, and you'll see why I chose 4kb. (DAMN, I wish I could remember what I fixed?!) Debbie Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.