Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 15 years ago

#1308 closed enhancement (duplicate)

Remove avatar Ids to allow w3c validated pages

Reported by: sbrajesh's profile 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)

Change History (1)

#1 @r-a-y
15 years ago

  • Resolution set to duplicate
  • Status changed from new to closed

I've closed this ticket and created a patch here:
http://trac.buddypress.org/ticket/1328

Note: See TracTickets for help on using tickets.