Skip to:

Opened 14 years ago

Closed 12 years ago

#1299 closed enhancement (fixed)

site wide activity widget should aware "rel" tag

Reported by: levinng's profile levinng Owned by: masonjames's profile masonjames
Milestone: 1.5 Priority: minor
Severity: Version:
Component: Activity Keywords: has-patch


I'd like to suggest site wide activity widget should allow "rel" tag for images,
it'll open up for image display plugin to work properly, such as lightbox or thickbox.

Attachments (1)

reltag.diff (568 bytes) - added by masonjames 12 years ago.
Added 'rel' attribute to the activity filter

Download all attachments as: .zip

Change History (10)

#1 @apeatling
14 years ago

  • Milestone set to 1.2
  • Type changed from enhancement to defect

Will add support for this in 1.2.

#2 @apeatling
13 years ago

  • Keywords needs-patch added

Need a patch for this.

#3 @apeatling
13 years ago

  • Milestone changed from 1.2 to 1.2.1
  • Type changed from defect to enhancement

Bumping to 1.2.1.

#4 @boonebgorges
12 years ago

  • Component set to Core
  • Milestone changed from 1.3 to 1.4

There is no sitewide activity widget in BP anymore, though I assume the issue is more broadly relevant.

Punting to 1.4 in the absence of a patch. If anyone wants to step up, please feel free to bump this back to 1.3.

#5 @DJPaul
12 years ago

I assume this would just involve adding the rel tag to the bp_activity_filter_kses() filter? Easy props for someone if they would like to patch it.

#6 @masonjames
12 years ago

  • Owner set to masonjames
  • Status changed from new to accepted

12 years ago

Added 'rel' attribute to the activity filter

#7 @masonjames
12 years ago

  • Keywords has-patch added; needs-patch removed
  • Resolution set to fixed
  • Status changed from accepted to closed

Here's my first patch. Feel free to let me know if I've done it incorrectly :)

#8 @boonebgorges
12 years ago

  • Component changed from Core to Activity
  • Milestone changed from 1.4 to 1.3
  • Priority changed from major to minor
  • Resolution fixed deleted
  • Status changed from closed to reopened

Thanks for the patch, I'll have a look shortly.

#9 @boonebgorges
12 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

Looks like this is already done in the trunk, as of r3494.

Note: See TracTickets for help on using tickets.