search

mozilla / hotdish

Explorations on being together in the browser
Mozilla Public License 2.0
111 stars 13 forks source link

The null avatar is terrible #264

Open ianb opened 10 years ago

ianb commented 10 years ago

When someone doesn't set their avatar, it's just white. Not even a generic, just blank and white. We use the avatar for identification in a lot of places. Truly forcing people to take a picture for their avatar is technically somewhat difficult.

ianb commented 10 years ago

In package.json you can set defaults for things like the avatar pref. If there's a public location to point to for the avatar URL that will work, otherwise if it's local we'll have to tweak it (maybe in the lib/identity module).

aarondruck commented 10 years ago

@ianb adding default avatar - let me know if this works

default-avatar default-avatar 2x

ianb commented 10 years ago

Since we are using these avatars for identification in a bunch of places (with no names) maybe we should have a few different colors of avatar?

I put in a site/ directory which I'll setup to be uploaded to https://togetherjs.com/hotdish/ – these avatars could go in there, and then we'd create a list of the full URLs and randomly pick one in lib/identity.js if there is no other avatar assigned to the pref.

aarondruck commented 10 years ago

@ianb added in the avatars, can you plug them in? or direct me?

ianb commented 10 years ago

@aarondruck if you want to try to set them up, it's not too hard a task. I've uploaded the avatars, at https://togetherjs.com/hotdish/avatars/avatar-green.png (etc).

You'll want to look at lib/identity.js – and check if myprefs.avatar is set, and if not then pick a random avatar.