Allow Multiple activity loop on same page work properly

[sbrajesh]

At the moment, it is possible to create multiple activity loops and have them on same page.
The problem lies the way current js is implemented it. The code does not differentiate between the different loops and if you do any action on one of the loop items, the other loop gets affected.
For example:-

• Load more items in first loop, all other loops will have the same done
• Filtering of activity using the selectboxes, all other loops gte filtered
• same goes for comment/reply etc

The problem lies with the javascipt code, It should be scoped to the the loop.

Please find a patch attached. I will be happy to help get it working, please let me know your suggestions for improving the patch.

[slaFFik]

Thanks for the patch, @sbrajesh!

That's for the activity, and what about groups, members, blogs etc loops?
Shouldn't we create something universal?

[DJPaul]

I'd personally prefer to see this sort of testing and enhancement to be done with BP-Nouveau, not BP-Default.

@slaFFik don't let perfect become the enemy of better.

[r-a-y]

Thanks for the patch, @sbrajesh. Not as bad as I thought it was going to be :)

How does this affect the activity heartbeat though? Does heartbeat run correctly on multiple activity loops?

[DJPaul]

This continues to be neat, but as Nouveau is legitimately close, let's not enhance Legacy any more. Thank you for your effort here @sbrajesh and sorry we didn't ship it earlier.