Thread Rating:
  • 2 Vote(s) - 4 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Identicon Avatars
Looks like a nice idea Paul. I definitely agree with the idea of caching though.
1. avatars (it seems they have transparency) looks bad on dark backgrounds:

[Image: 1323845017-clip-23kb.png] [Image: 1323848010-clip-17kb.png]

I suppose white color must be fully transparent. But now there some "noisy holes" there)

2. identicons don't displays in profiles, memberlist, etc...
This user has been denied support. This user has been denied support.
^Paul, have you added the hooks in memprofile and all as well so they show up there? If not, you might wanna add it for next versions.
3. Registered users with same IP adress have same avatar. It's also not good. I think each registered user must have unique avatar. You can modify hash based on user's ID.
I will be making some massive changes in the next version. This really was just a beta release.
-Paul H.

Cogisne lingua latina?
I'm testing my cache system out with my avatar here on MyBB. It should work.
-Paul H.

Cogisne lingua latina?
It's works.

[Image: 1323936923-clip-25kb.png]

Btw... is there any settings to control complexity of images? Now it uses 3x3 matrix to produce images. What about 5x5? )
You would have to edit the ./identicon.php file. It's not my script, though, so I don't know how to do it off the top of my head.
-Paul H.

Cogisne lingua latina?
No problems! That's not necessary.
I made my avatars based on user's ID. It solves 2 problems:

1. Now each registered user have it's own unique avatar.
2. No more security risk of exposing user IP addresses.

To do it, open identicon_avatar.php, find:

if($post['uid']) {$ip = $post['regip'];} else {$ip = $post['ipaddress'];}

and change it to:

if($post['uid']) {$ip = $post['uid'];} else {$ip = $post['ipaddress'];}

Forum Jump:

Users browsing this thread: 1 Guest(s)