Skip to:

Opened 6 years ago

Closed 6 years ago

#7773 closed defect (bug) (fixed)

BP Nouveau: new activity pages should be inserted into the exisiting activity list

Reported by: imath's profile imath Owned by: imath's profile imath
Milestone: 3.0 Priority: normal
Severity: normal Version:
Component: Templates Keywords: has-patch commit


I'm not sure my summary is really clear :)

This screenshot will help me to explain what i think is an issue :

Each time you load more activities, there's a new ul tag generated. As it's not the case in BP Legacy where new activities are inserted into the existing ul, i think it's an issue.

@hnla if it's on purpose to have this separation between batch of per_page activities, do not hesitate to close this ticket ;)

Attachments (1)

7773.patch (1.6 KB) - added by imath 6 years ago.

Download all attachments as: .zip

Change History (4)

6 years ago

#1 @DJPaul
6 years ago

Go for it

#2 @imath
6 years ago

  • Keywords commit added
  • Owner set to imath
  • Status changed from new to assigned

👍 thanks for your feedback

#3 @imath
6 years ago

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

In 12033:

BP Nouveau: Load older activities (load-more) in the same "ul" list.

The "ul" container of the activity stream loop needs to be exclude of paginated results to avoid having each page in a different container.

Fixes #7773

Note: See TracTickets for help on using tickets.