[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