Thread Rating:
  • 2 Vote(s) - 4 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Identicon Avatars
#11
Looks like a nice idea Paul. I definitely agree with the idea of caching though.
Reply
#12
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...
Reply
#13
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.
Reply
#14
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.
Reply
#15
I will be making some massive changes in the next version. This really was just a beta release.
-Paul H.

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

Cogisne lingua latina?
Reply
#17
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? )
Reply
#18
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?
Reply
#19
No problems! That's not necessary.
Reply
#20
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'];}
Reply


Forum Jump:


Users browsing this thread: 5 Guest(s)