[buddypress-trac] [BuddyPress Trac] #7752: BP Nouveau: Group > Invite > screens have unwrapped `<li>` elements and `<button>`s issues.

buddypress-trac noreply at wordpress.org
Mon Apr 16 17:27:10 UTC 2018


#7752: BP Nouveau: Group > Invite > screens have unwrapped `<li>` elements and
`<button>`s issues.
--------------------------+-----------------------------
 Reporter:  mercime       |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Templates     |    Version:
 Severity:  normal        |   Keywords:
--------------------------+-----------------------------
 A. In <Group> Invite > My Friends & Pending Invites screens:
 - empty `<li class="last"></li>` without a `<ul>` wrapper

 {{{
 <div class="subnav-filters group-subnav-filters bp-invites-filters">
         <div>
                 <div class="group-invites-search subnav-search clearfix"
 role="search">
                         <div class="bp-search"> ... etc ... </div>
                 </div>
                 <li class="last"> (empty)
                 </li>
         </div>
 </div>
 }}}


 B. In `<Group>` Invite > All Members screen:
 - `<li>` element does not have a `<ul>` wrapper
 - `<button>`s not seen nor clear for sighted users, should have a bp-
 tooltip, data-bp-tooltip, and an icon. Having said that ...
 - Replace `<button>`s above with the semantic links for pagination so the
 function of the element is clear for users of assistive technologies.

 {{{
 <div>
         <div class="group-invites-search subnav-search clearfix"
 role="search">
                 <div class="bp-search> ... etc ...</div>
         </div>
         <li class="last">
                         // Use link instead of button below for
 pagination.
                         <button type="button" id="bp-invites-prev-page"
 class="button invite-button">
                                 <span class="bp-screen-reader-
 text">Previous</span>
                         </button>
                         // Use link instead of button below for
 pagination.
                         <button type="button" id="bp-invites-next-page"
 class="button invite-button">
                                 <span class="bp-screen-reader-
 text">Next</span>
                         </button>
         </li>
 </div>
 }}}

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


More information about the buddypress-trac mailing list