Skip to:
Content

BuddyPress.org

Changeset 6555


Ignore:
Timestamp:
12/01/2012 02:02:39 AM (12 years ago)
Author:
johnjamesjacoby
Message:

Revert recent activity ajax hide/fade changes, and fix issues with new item hide/fade causing the entire node to flicker. For bp-default/bp-legacy.

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/bp-templates/bp-legacy/js/buddypress.js

    r6552 r6555  
    439439                /* Check for errors and append if found. */
    440440                if ( response[0] + response[1] == '-1' ) {
    441                     form.append( response.substr( 2, response.length ) ).fadeIn( 200 );
     441                    form.append( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    442442                } else {
    443443                    form.fadeOut( 200, function() {
     
    450450                        }
    451451
    452                         form.parent().children('ul').append(response).fadeIn( 200 );
     452                        form.parent().children('ul').append( jq( response ).hide().fadeIn( 200 ) );
    453453                        form.children('textarea').val('');
    454454                        form.parent().parent().addClass('has-comments');
     
    497497                /* Check for errors and append if found. */
    498498                if ( response[0] + response[1] == '-1' ) {
    499                     comment_li.prepend( response.substr( 2, response.length ) ).hide().fadeIn( 200 );
     499                    comment_li.prepend( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    500500                } else {
    501501                    var children = jq( '#' + comment_li.attr('id') + ' ul' ).children('li');
     
    545545                // Check for errors and append if found.
    546546                if ( response[0] + response[1] == '-1' ) {
    547                     comment_li.prepend( response.substr( 2, response.length ) ).hide().fadeIn( 200 );
     547                    comment_li.prepend( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    548548
    549549                } else {
  • trunk/bp-themes/bp-default/_inc/global.js

    r6553 r6555  
    430430            }
    431431
    432             jq.post( ajaxurl, ajaxdata,
    433                 function(response)
    434                 {
    435                     target.removeClass('loading');
    436 
    437                     /* Check for errors and append if found. */
    438                     if ( response[0] + response[1] == '-1' ) {
    439                         form.append( response.substr( 2, response.length ) ).fadeIn( 200 );
    440                     } else {
    441                         form.fadeOut( 200,
    442                             function() {
    443                                 if ( 0 == form.parent().children('ul').length ) {
    444                                     if ( form.parent().hasClass('activity-comments') )
    445                                         form.parent().prepend('<ul></ul>');
    446                                     else
    447                                         form.parent().append('<ul></ul>');
    448                                 }
    449 
    450                                 form.parent().children('ul').append(response).fadeIn( 200 );
    451                                 form.children('textarea').val('');
    452                                 form.parent().parent().addClass('has-comments');
     432            jq.post( ajaxurl, ajaxdata, function(response) {
     433                target.removeClass('loading');
     434
     435                /* Check for errors and append if found. */
     436                if ( response[0] + response[1] == '-1' ) {
     437                    form.append( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
     438                } else {
     439                    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>');
     443                            } else {
     444                                form.parent().append('<ul></ul>');
    453445                            }
    454                             );
    455                         jq( 'form#' + form.attr('id') + ' textarea').val('');
    456 
    457                         /* Increase the "Reply (X)" button count */
    458                         jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html( Number( jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html() ) + 1 );
    459                     }
    460 
    461                     jq(target).prop("disabled", false);
    462                 });
     446                        }
     447
     448                        form.parent().children('ul').append( jq( response ).hide().fadeIn( 200 ) );
     449                        form.children('textarea').val('');
     450                        form.parent().parent().addClass('has-comments');
     451                    } );
     452
     453                    jq( 'form#' + form.attr('id') + ' textarea').val('');
     454
     455                    /* Increase the "Reply (X)" button count */
     456                    jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html( Number( jq('li#activity-' + form_id[2] + ' a.acomment-reply span').html() ) + 1 );
     457                }
     458
     459                jq(target).prop("disabled", false);
     460            });
    463461
    464462            return false;
     
    496494                /* Check for errors and append if found. */
    497495                if ( response[0] + response[1] == '-1' ) {
    498                     comment_li.prepend( response.substr( 2, response.length ) ).fadeIn( 200 );
     496                    comment_li.prepend( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    499497                } else {
    500498                    var children = jq( 'li#' + comment_li.attr('id') + ' ul' ).children('li');
     
    544542                // Check for errors and append if found.
    545543                if ( response[0] + response[1] == '-1' ) {
    546                     comment_li.prepend( response.substr( 2, response.length ) ).fadeIn( 200 );
     544                    comment_li.prepend( jq( response.substr( 2, response.length ) ).hide().fadeIn( 200 ) );
    547545
    548546                } else {
Note: See TracChangeset for help on using the changeset viewer.