Skip to:
Content

BuddyPress.org

Opened 16 years ago

Closed 16 years ago

Last modified 10 years ago

#2502 closed defect (bug) (fixed)

JS: bp_dtheme_hide_comments() doesn't work when using an AJAX request

Reported by: r-a-y Owned by:
Priority: normal Milestone: 1.2.6
Component: Templates Version:
Severity: Keywords: has-patch
Cc:

Description

In the bp-default theme, when there are more than five root activity comments for an activity update, the latest five comments will be shown and the rest will be hidden.

However, when you click on any AJAX tab (eg. "All Members"), this doesn't work as expected.

The problem is the order in which bp_dtheme_hide_comments() is fired. The attached patch fixes this.

Btw, we need a new component in Trac specifically for the default theme ;)

Attachments (1)

2502.001.patch (730 bytes ) - added by r-a-y 16 years ago.

Download all attachments as: .zip

Change History (4)

@r-a-y
16 years ago

#1 @johnjamesjacoby
16 years ago

  • Component CoreTheme

'Theme' component added. Thanks for the patch r-a-y!

#2 @johnjamesjacoby
16 years ago

  • Resolutionfixed
  • Status newclosed

(In [3130]) Fixes #2502 props r-a-y

#3 @DJPaul
10 years ago

  • Component Appearance - Template PartsTemplates
Note: See TracTickets for help on using tickets.