Opened 11 years ago
Closed 11 years ago
#5452 closed defect (bug) (fixed)
Order notifications by date
Reported by: | mpa4hu | Owned by: | |
---|---|---|---|
Milestone: | 2.0 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Toolbar & Notifications | Keywords: | ui-feedback reporter-feedback |
Cc: |
Description
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.
Change History (3)
#2
@
11 years ago
- Keywords reporter-feedback added
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?
In 8153: