Opened 15 years ago
Closed 15 years ago
#1308 closed enhancement (duplicate)
Remove avatar Ids to allow w3c validated pages
Reported by: | sbrajesh | Owned by: | |
---|---|---|---|
Milestone: | 1.1.3 | Priority: | minor |
Severity: | Version: | ||
Component: | Keywords: | w3c, avatar | |
Cc: |
Description
well,since avatars have ids and say when the same avatar appears multiple times on a page,the page gets multiple element with same Ids.
It creates troubles in validating the themes.
A better solution will be to just remove the id from the avatar image.
We can edit bp_core_fetch_avatar and make it possible.
There can be a simple fix by changing this line in bp_core_fetch_avatar.
return apply_filters( 'bp_core_fetch_avatar', "<img src='{$avatar_url}' alt='{$alt}' id='{$css_id}' class='{$class}'{$html_width}{$html_height} />", $params );
to
return apply_filters( 'bp_core_fetch_avatar', "<img src='{$avatar_url}' alt='{$alt}' class='{$class}'{$html_width}{$html_height} />", $params );
however,if you want to disable computation of ids(as it is not required now,you will have to edit a few other lines too)
I've closed this ticket and created a patch here:
http://trac.buddypress.org/ticket/1328