search

gnocode / noladex.org

People of New Orleans
http://www.noladex.org
21 stars 10 forks source link

disallow animated profile pics #39

Closed drafnel closed 12 years ago

drafnel commented 13 years ago

See Adam Clarke's profile pic of a "Dancing_kid" animated gif. I don't think this is something we want to allow.

Should we convert the image? or detect it at the time it is submitted?

Can we do something as simple as:

convert -flatten $orig ${orig}_noladex.jpg

When the above is applied to an animated gif, the output will be a jpeg of the first frame of the animated gif sequence. For any other image format, the output will be a jpeg version of the original.

mshwery commented 13 years ago

Or at least put a max limit on file size - his picture is 1.7mb alone.

Matt

On Thu, Jul 28, 2011 at 10:34 AM, drafnel < reply@reply.github.com>wrote:

See Adam Clarke's profile pic of a "Dancing_kid" animated gif. I don't think this is something we want to allow.

Should we convert the image? or detect it at the time it is submitted?

Can we do something as simple as:

convert -flatten $orig ${orig}_noladex.jpg

When the above is applied to an animated gif, the output will be a jpeg of the first frame of the animated gif sequence. For any other image format, the output will be a jpeg version of the original.

Reply to this email directly or view it on GitHub: https://github.com/gnocode/noladex.org/issues/39

drafnel commented 13 years ago

His isn't the largest pic. The largest is 2.6 MB. There are 3 pics over 2MB and 5 others over 1MB (not incl. Dancing_kid). So, we should probably do something about these very large images aside from the animated gif issue. I'll add a new entry for this.

-Brandon

On Thu, Jul 28, 2011 at 10:39 AM, mshwery reply@reply.github.com wrote:

Or at least put a max limit on file size - his picture is 1.7mb alone.

Matt

On Thu, Jul 28, 2011 at 10:34 AM, drafnel < reply@reply.github.com>wrote:

See Adam Clarke's profile pic of a "Dancing_kid" animated gif.  I don't think this is something we want to allow.

Should we convert the image? or detect it at the time it is submitted?

Can we do something as simple as:

  convert -flatten $orig ${orig}_noladex.jpg

When the above is applied to an animated gif, the output will be a jpeg of the first frame of the animated gif sequence.  For any other image format, the output will be a jpeg version of the original.

Reply to this email directly or view it on GitHub: https://github.com/gnocode/noladex.org/issues/39

Reply to this email directly or view it on GitHub: https://github.com/gnocode/noladex.org/issues/39#issuecomment-1673627

vaxinate commented 13 years ago

please change issue name to "disallow fun"

the site is using s3 to store images, a 1.7 MB gif adds pretty much nothing to storage and bandwidth costs. I understand it could cause a load time issue for mobile devices or people with very slow internet connections, but given that the whole site is loading images, animated gifs hardly seem like a problem.

If we're going to ban animated gifs, please ban all images that aren't a picture of the person who the profile belongs to. I want to have a profile that represents me, and I'm an animated gif kind of guy. Please don't remove my abilityto express myself because it's not "professional" enough for you.

I also understand that I do not run this site and it's up to whoever does, but I wanted to give my 2 cents since i seem to have been called out.

poemdexter commented 13 years ago

drafnel, is there a reason behind "I don't think this is something we want to allow." ? Bandwidth concern? Professionalism concern? Design concern?

cafink commented 13 years ago

I think we should require all profile images to be animated GIFs. It would liven up the site a lot!

joeellis commented 13 years ago

I agree with Carl; not on any particular merit or point, but just because Carl is awesome in general.

I also think if you choose an image that goes over a certain limit, say (1.5mb?), then you are required to pay Barrett Conrad $1 for the Barrett-graciously-lent-his-credit-card-for-this-so-we-could-have-nice-things tax

drafnel commented 13 years ago

Professionalism and design, I'd say are the main concerns. I imagine the site could become a big flashing mess if enough people decided they wanted animated profile pics, each one animating at a different frame rate. I also find that animations draw my attention straight to them (that's why advertisers use them) and I think the site would be better if that didn't happen to people. Plus my non-objective opinion is that the purely static images make for a simpler, classier site. On Jul 28, 2011 4:21 PM, "poemdexter" < reply@reply.github.com> wrote:

vaxinate commented 13 years ago

drafnel,

I'm going to have to non-objectively disagree. It's all in good fun. The reason I used the image in the first place was because the site felt so stuffy. If we don't let people express themselves, then what's the point of this whole site? Are we really representing the spirit of New Orleans if we turn it into something all about professionalism and class? If it really becomes a problem, or a lot of people are complaining about it, then sure, let's remove the ability, but I just don't think its hurting anything and i think it adds some fun to the site. For now, let's leave this issue open and see if anyone else opposed to animated images weighs in.

daybreaker commented 13 years ago

I dont mind animated images too much, as long as theyre not overly distracting, and are used sparingly.... My main concern would be file size affecting load times. Even though this could be mitigated some with an infinite scroll plugin, whenever the scroll ajax hits one of those large images, it would cause a huge delay in loading the next few rows.

Does paperclip allow you to set a maximum file size? This would help limit animated gifs as well, in that to hit 300x300, most gifs would wind up exceeding the 1.5mb (or whatever is decided upon) limit.

joeellis commented 13 years ago

Yes, there is a validation built in on the README page for the paperclip gem on how to set validations for attachment sizes

Joe Ellis squarefour | www.squarefour.net 504.267.9321 joe@squarefour.net

On Friday, July 29, 2011 at 3:01 PM, daybreaker wrote:

I dont mind animated images too much, as long as theyre not overly distracting, and are used sparingly.... My main concern would be file size.

Does paperclip allow you to set a maximum file size? This would help limit animated gifs as well, in that to hit 300x300, most gifs would wind up exceeding the 1.5mb (or whatever is decided upon) limit.

Reply to this email directly or view it on GitHub: https://github.com/gnocode/noladex.org/issues/39#issuecomment-1684211

poemdexter commented 13 years ago

If there's ever a time where we get rid of the animated gif completely, we might as well take it to it's logical step and just get rid of non human face images. It's just as jarring to hit a picture of a tree as it is to see a kid living the high life on camera.

mshwery commented 12 years ago

Seems like we have no direction on this issue, but the general consensus is to leave it alone. Closing this ticket. If this proves to be an issue, open this ticket back up.