[buddypress-trac] [BuddyPress Trac] #6005: No-js bulk deletion of messages

[buddypress-trac]

#6005: No-js bulk deletion of messages
----------------------------------------+------------------
 Reporter:  lakrisgubben                |       Owner:
     Type:  enhancement                 |      Status:  new
 Priority:  normal                      |   Milestone:  2.2
Component:  Messaging                   |     Version:
 Severity:  normal                      |  Resolution:
 Keywords:  needs-refresh dev-feedback  |
----------------------------------------+------------------

Comment (by hnla):

 Replying to [comment:14 boonebgorges]:
 > > I mean you've got a screen with 10 messages on it, do you really need
 a way to toggle only the read/unread?
 >
 > I agree. I'd remove it. Let's hear what others have to say.
 Although I do quite like the option to be able to select one or the other
 en masse, I agree it's too cluttered to have too many selects especially
 where we only show quite a limited max number of messages per screen; so
 think I would have to agree too.

 > > Why do we only show the unread count for the messages?
 >
 > I think slaFFik's answer is probably partly right - it provides an a11y-
 friendly way to see what's unread. But I think the a11y concerns could
 probably be accomplished in other ways. I personally am not attached to
 the count boxes, and I do think they clutter the interface a bit. I'd like
 to hear what hnla in particular thinks.
 >

 This is a slightly more involved issue, the backgrounds are, as slaFFik
 point out, simply visual cues carrying no real semantic meaning, nothing
 wrong with that, but it does mean we need a further identifier as to the
 status of the information being conveyed. The numbering goes some way
 towards this, however in its self again actually carries little actual
 meaning it's a number! we only generally now to what this pertains through
 experience; someone seeing this for the first time would be forgiven for
 looking blank.

 Properly describing the table (part of the exercise and point of this
 ticket to match to notifications) with TH elements will allow for a clear
 'Unread' column heading, added to this  and given we count the number of
 unreads within the thread I would suggest we also add a '0' count for no
 unreads to keep the column reading in a sane manner "''Unread: 0''" rather
 than suddenly being devoid of content.

 If we want to take things a step further we could consider adding a span
 with screen reader text e.g:
 `<td><span class="unread-count">2</span><span class="bp-screen-reader-
 text">Unread messages</span></td>`

 I feel with a correctly described table replete with TH headers and fully
 scope attributed we cover things though.

 > > Which brings me on to the bold text on unread messages
 >
 > I'm kinda indifferent, but I agree that other email clients do it this
 way, so I lean toward leaving it the way it is.

 I would agree it was unnecessary and BP needn't have been doing this which
 is tantamount to visual styling with no particular purpose, but equally am
 indifferent to changing it it's been out in the wild now for some time.
 >
 > > I think we should have a "mark read/mark unread" link along with the
 delete link next to each thread, just as we have with notifications.
 Anyone opposed to this?
 >
 > #4531 refers to the bulk actions, and has now been resolved. I'm not
 opposed in theory to adding per-thread links, though I don't want it to
 introduce too much clutter to the page. I'd be glad to see a patch or a
 mockup.

 Agreed  not opposed either, slightly in favour I think, but best to see
 this in action so to speak to decide if it feels too cluttered.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6005#comment:16>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac