Order notifications by date

[mpa4hu]

For now notifications are ordered by component.

I had little hack which was including date with every notification and on client side I was resorting them with javascript.

That was wrong of course, mainly because I had core hacks to include date. And it was really hard to update that with every new update (glad I got rid of that)

But this is still a problem. Nor order it by date nor it should be visible (separated) that these notifications are coming from different components and thats why you got this in this order.

[boonebgorges]

In 8153:

Add test for sort_order and order_by date in notifications component.

See #5452

[boonebgorges]

Notifications can already be queried by date. Here's a unit test that shows how: ​https://buddypress.trac.wordpress.org/browser/trunk/tests/testcases/notifications/class-bp-notifications-notification.php#L119

Are you suggesting something in the UI? What would it look like?