[buddypress-trac] [BuddyPress Trac] #6710: notices-loop

[buddypress-trac]

#6710: notices-loop
---------------------------------------+-----------------------------
 Reporter:  danbp                      |      Owner:
     Type:  defect (bug)               |     Status:  new
 Priority:  normal                     |  Milestone:  Awaiting Review
Component:  Component - Notifications  |    Version:
 Severity:  normal                     |   Keywords:
---------------------------------------+-----------------------------
 BP 2.4 rc1
 Twenty Sixteen

 - the notice delete message button use '''x''' instead of Delete Message.
 This x is in the same column as activate/deactivate status and isn't
 pertinent (at least to me) when it comes as second line under the action
 name. Should at least be in a separate column if the aim is to gain some
 place. That said i ignore why this part is styled like this, as the
 similar one for message management is slightly different, and much better.

 The template itself looks awful on Twenty Sixteen and contains some
 useless tags like strong and a mysterious empty td of 1% !

 Here the code with real content of what i'm speaking about.


 {{{
 <tbody><tr id="notice-2" class="">
    <td width="1%"></td>
       <td width="38%">
         <strong>This is a notice to all users.</strong>
         <p>This is a notice test to all users.</p>
       </td>
       <td width="21%">
        <strong></strong>
        <span class="activity">Sent: 2 November 2015 at 17 h 44 min</span>
       </td>
       <td width="10%">
        <a class="button"
 href="http://wpbeta/members/daniel/messages/notices/deactivate/2/?_wpnonce=227dc95ecc">Deactivate</a>
        <a class="button"
 href="http://wpbeta/members/daniel/messages/notices/delete/2?_wpnonce=ded41a06e5"
 title="Delete Message">x</a>
       </td>
    </tr>
 </tbody>
 }}}

 Concerned file is [https://buddypress.trac.wordpress.org/browser/trunk/src
 /bp-templates/bp-legacy/buddypress/members/single/messages/notices-
 loop.php/ notices-loop]

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6710>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac