You are not logged in. Please login or register.
PunBB Forums → PunBB 1.3 troubleshooting → Function for displaying an avatar
I believe it would be a good idea to have a single function that returns the proper HTML code to display an avatar, and have a hook in that function.
This would allow extensions to modify the displaying of avatars, for example adding image filetypes.
Mmm, that makes sense. But if we do that, we should also move deleting avatars into its own function for the same reason (especially since we do it in several places).
I don't get it. The proper html code for an avatar is an img tag. What else is needed?
Well, you generate it. You check which extension has been used and return the proper img tag on basis of that.
Right now, the same code is being used in various places. This would also make it easier to display avatars at different locations (eg. PMs.)
And indeed Smartys.
Edit: What Bekko said
If you mean we can delete all that image type/size fetching stuff from profile and viewtopic then I'm all for it.
Exactly.
I like this idea... 
PunBB Forums → PunBB 1.3 troubleshooting → Function for displaying an avatar
Powered by PunBB, supported by Informer Technologies, Inc.