[buddypress-trac] [BuddyPress Trac] #5513: Bulk Notifications Management
buddypress-trac
noreply at wordpress.org
Mon Nov 10 22:20:30 UTC 2014
#5513: Bulk Notifications Management
---------------------------+------------------
Reporter: colabsadmin | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 2.2
Component: Notifications | Version: 2.0
Severity: normal | Resolution:
Keywords: needs-patch |
---------------------------+------------------
Comment (by lakrisgubben):
Replying to [comment:8 boonebgorges]:
> This is the way to do it :) It looks like we *don't* do this in the case
of messages (which is the other obvious place where we do bulk
management), but we should. I'd build it this way first, and hold off on
the AJAX for the time being.
Cool, just wanted to be sure that I didn't break bp-protocoll. :) If this
patch gets accepted sometime maybe the next mission would be to fix this
for messages as well.
The new patch puts the checkboxes to the left, (hopefully) has better
function names, gets rid of most of the js (only saving the part for
selecting all), hides the select all if body.no-js and only uses page
refresh to post notification ids to bulk manage.
If you're wondering about
{{{
#buddypress #notifications-bulk-management {
clear: left;
}
}}}
it's because firefox flipped out if the form wasn't cleared and put the
table to the right of the whole content area. :/
btw. thanks for a great talk at WCSF! :)
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/5513#comment:9>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list