Skip to:
Content

BuddyPress.org

Changeset 7002


Ignore:
Timestamp:
05/06/2013 12:49:35 AM (13 years ago)
Author:
boonebgorges
Message:

Update 'Show all x comments' count after creating or deleting activity comments

On activity permalink pages, the comment list is collapsed if the item has more
than 6 responses, and a link is appended that reads 'Show all x comments'. This
changeset introduces additional JavaScript to update this count when a new
activity comment is created, or an existing one is deleted, via AJAX.

In the process of making the change, the logic for constructing the 'Show all
x comments' string was changed, so that it's now properly translatable.

Also cached a JS variable that was being used many times, to avoid unnecessary
DOM traversal.

Fixes #4844

Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/bp-templates/bp-legacy/buddypress-functions.php

    r6867 r7002  
    253253                        'rejected'          => __( 'Rejected', 'buddypress' ),
    254254                        'show_all_comments' => __( 'Show all comments for this thread', 'buddypress' ),
     255                        'show_x_comments'   => __( 'Show all %d comments', 'buddypress' ),
    255256                        'show_all'          => __( 'Show all', 'buddypress' ),
    256257                        'comments'          => __( 'comments', 'buddypress' ),

  • trunk/bp-templates/bp-legacy/js/buddypress.js

    r7000 r7002  
    444444                                        form.append( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    445445                                } else {
     446                                        var activity_comments = form.parent();
    446447                                        form.fadeOut( 200, function() {
    447                                                 if ( 0 == form.parent().children('ul').length ) {
    448                                                         if ( form.parent().hasClass('activity-comments') ) {
    449                                                                 form.parent().prepend('<ul></ul>');
     448                                                if ( 0 == activity_comments.children('ul').length ) {
     449                                                        if ( activity_comments.hasClass('activity-comments') ) {
     450                                                                activity_comments.prepend('<ul></ul>');
    450451                                                        } else {
    451                                                                 form.parent().append('<ul></ul>');
     452                                                                activity_comments.append('<ul></ul>');
    452453                                                        }
    453454                                                }
    … …  
    456457                                                var the_comment = jq.trim( response );
    457458
    458                                                 form.parent().children('ul').append( jq( the_comment ).hide().fadeIn( 200 ) );
     459                                                activity_comments.children('ul').append( jq( the_comment ).hide().fadeIn( 200 ) );
    459460                                                form.children('textarea').val('');
    460                                                 form.parent().parent().addClass('has-comments');
     461                                                activity_comments.parent().addClass('has-comments');
    461462                                        } );
    462463                                        jq( '#' + form.attr('id') + ' textarea').val('');
    … …  
    464465                                        /* Increase the "Reply (X)" button count */
    465466                                        jq('#activity-' + form_id[2] + ' a.acomment-reply span').html( Number( jq('#activity-' + form_id[2] + ' a.acomment-reply span').html() ) + 1 );
     467
     468                                        // Increment the 'Show all x comments' string, if present
     469                                        var show_all_a = activity_comments.find('.show-all').find('a');
     470                                        if ( show_all_a ) {
     471                                                var new_count = jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html();
     472                                                show_all_a.html( BP_DTheme.show_x_comments.replace( '%d', new_count ) );
     473                                        }
    466474                                }
    467475
    … …  
    519527                                        var new_count = count_span.html() - ( 1 + child_count );
    520528                                        count_span.html(new_count);
     529                                       
     530                                        // Change the 'Show all x comments' text
     531                                        var show_all_a = comment_li.siblings('.show-all').find('a');
     532                                        if ( show_all_a ) {
     533                                                show_all_a.html( BP_DTheme.show_x_comments.replace( '%d', new_count ) );
     534                                        }
    521535
    522536                                        /* If that was the last comment for the item, remove the has-comments class to clean up the styling */
    … …  
    14071421                                jq(this).toggle();
    14081422
    1409                                 if ( !i )
    1410                                         jq(this).before( '<li class="show-all"><a href="#' + parent_li.attr('id') + '/show-all/" title="' + BP_DTheme.show_all_comments + '">' + BP_DTheme.show_all + ' ' + comment_count + ' ' + BP_DTheme.comments + '</a></li>' );
     1423                                if ( !i ) 
     1424                                        jq(this).before( '<li class="show-all"><a href="#' + parent_li.attr('id') + '/show-all/" title="' + BP_DTheme.show_all_comments + '">' + BP_DTheme.show_x_comments.replace( '%d', comment_count ) + '</a></li>' );
    14111425                        }
    14121426                });

  • trunk/bp-themes/bp-default/_inc/global.js

    r7000 r7002  
    437437                                        form.append( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    438438                                } else {
     439                                        var activity_comments = form.parent();
    439440                                        form.fadeOut( 200, function() {
    440                                                 if ( 0 == form.parent().children('ul').length ) {
    441                                                         if ( form.parent().hasClass('activity-comments') ) {
    442                                                                 form.parent().prepend('<ul></ul>');
     441                                                if ( 0 == activity_comments.children('ul').length ) {
     442                                                        if ( activity_comments.hasClass('activity-comments') ) {
     443                                                                activity_comments.prepend('<ul></ul>');
    443444                                                        } else {
    444                                                                 form.parent().append('<ul></ul>');
     445                                                                activity_comments.append('<ul></ul>');
    445446                                                        }
    446447                                                }
    … …  
    449450                                                var the_comment = jq.trim( response );
    450451
    451                                                 form.parent().children('ul').append( jq( the_comment ).hide().fadeIn( 200 ) );
     452                                                activity_comments.children('ul').append( jq( the_comment ).hide().fadeIn( 200 ) );
    452453                                                form.children('textarea').val('');
    453                                                 form.parent().parent().addClass('has-comments');
     454                                                activity_comments.parent().addClass('has-comments');
    454455                                        } );
    455456
    … …  
    458459                                        /* Increase the "Reply (X)" button count */
    459460                                        jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html( Number( jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html() ) + 1 );
     461
     462                                        // Increment the 'Show all x comments' string, if present
     463                                        var show_all_a = activity_comments.find('.show-all').find('a');
     464                                        if ( show_all_a ) {
     465                                                var new_count = jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html();
     466                                                show_all_a.html( BP_DTheme.show_x_comments.replace( '%d', new_count ) );
     467                                        }
    460468                                }
    461469
    … …  
    513521                                        var new_count = count_span.html() - ( 1 + child_count );
    514522                                        count_span.html(new_count);
     523       
     524                                        // Change the 'Show all x comments' text
     525                                        var show_all_a = comment_li.siblings('.show-all').find('a');
     526                                        if ( show_all_a ) {
     527                                                show_all_a.html( BP_DTheme.show_x_comments.replace( '%d', new_count ) );
     528                                        }
    515529
    516530                                        /* If that was the last comment for the item, remove the has-comments class to clean up the styling */
    … …  
    13911405
    13921406                                if ( !i )
    1393                                         jq(this).before( '<li class="show-all"><a href="#' + parent_li.attr('id') + '/show-all/" title="' + BP_DTheme.show_all_comments + '">' + BP_DTheme.show_all + ' ' + comment_count + ' ' + BP_DTheme.comments + '</a></li>' );
     1407                                        jq(this).before( '<li class="show-all"><a href="#' + parent_li.attr('id') + '/show-all/" title="' + BP_DTheme.show_all_comments + '">' + BP_DTheme.show_x_comments.replace( '%d', comment_count ) + '</a></li>' );
    13941408                        }
    13951409                });

  • trunk/bp-themes/bp-default/functions.php

    r6726 r7002  
    160160                'rejected'          => __( 'Rejected', 'buddypress' ),
    161161                'show_all_comments' => __( 'Show all comments for this thread', 'buddypress' ),
     162                'show_x_comments'   => __( 'Show all %d comments', 'buddypress' ),
    162163                'show_all'          => __( 'Show all', 'buddypress' ),
    163164                'comments'          => __( 'comments', 'buddypress' ),
Note: See TracChangeset for help on using the changeset viewer.