Skip to:

Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#5467 closed enhancement (fixed)

Replace .parent() for .closest in buddypress.js

Reported by: hnla Owned by: boonebgorges
Milestone: 2.0 Priority: normal
Severity: normal Version:
Component: Templates Keywords: has-patch


In the same manner of previous tickets on this aspect another .parent().parent().parent() has caused issues with adjusted markup causing activity unfav in activity favourit view jumping it's slideUp() to the wrong parent node and hiding all the activity items rather than the activity-item in question.

Patch replaces .parent() with .closest('activity-item')

Attachments (2)

5467.patch (484 bytes) - added by hnla 6 years ago.
change .parent() to .closest() for activity unfav
5467.2.patch (485 bytes) - added by hnla 6 years ago.

Download all attachments as: .zip

Change History (5)

6 years ago

change .parent() to .closest() for activity unfav

6 years ago

#1 @boonebgorges
6 years ago

  • Milestone changed from Awaiting Review to 2.0

#2 @boonebgorges
6 years ago

  • Owner set to boonebgorges
  • Resolution set to fixed
  • Status changed from new to closed

In 8138:

Use closest() instead of parent() to hide unfavorited activity item

Ancestor-crawling makes for inflexible javascript, which makes for sad developers.

Fixes #5467

Props hnla

#3 @DJPaul
4 years ago

  • Component changed from Appearance - Template Parts to Templates
Note: See TracTickets for help on using tickets.