Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
01/27/2018 12:21:27 PM (7 years ago)
Author:
hnla
Message:

Nouveau: re-factor activity template loops main dir & members single

The loops JS inject all act content & loop messages into a ul construct regardless of whether there is true list loop content e.g no activity found or loading loop messages, single items are not a list, we should only show a ul if rendering loop content, this also impacts styles where messages look odd in a li element.

Commit moves the JS required hooks & 'data-bp-list="activity"' up a level to the wrapper element, re-factors the 'if bp_has_activities' to wrap only the ul which moves into the activity-loop.php template, moves the message functions out of the loops & removes unnecessary markup for messages.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-templates/bp-nouveau/buddypress/activity/index.php

    r11686 r11825  
    2929        <?php bp_nouveau_activity_hook( 'before_directory', 'list' ); ?>
    3030
    31         <div class="activity">
     31        <div id="activity-stream" class="activity" data-bp-list="activity">
    3232
    33             <ul id="activity-stream" class="activity-list item-list bp-list" data-bp-list="activity">
    34 
    35                 <li id="bp-ajax-loader"><?php bp_nouveau_user_feedback( 'directory-activity-loading' ); ?></li>
    36 
    37             </ul>
     33                <div id="bp-ajax-loader"><?php bp_nouveau_user_feedback( 'directory-activity-loading' ); ?></div>
    3834
    3935        </div><!-- .activity -->
    4036
    4137        <?php bp_nouveau_after_activity_directory_content(); ?>
     38
    4239    </div><!-- // .screen-content -->
    4340
Note: See TracChangeset for help on using the changeset viewer.