From 773839308c4bb3313c04aa02b75e95d342d51cc1 Mon Sep 17 00:00:00 2001 From: Rick Waldron Date: Fri, 17 Sep 2010 09:57:32 +0800 Subject: [PATCH] Moved .live(...) into $(function() {}) block - DOM logic should not be left to synchronously execute outside of a DOMContentLoaded closure --- public/javascripts/aspect-edit.js | 25 +++++++++++++------------ 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/public/javascripts/aspect-edit.js b/public/javascripts/aspect-edit.js index 86c61ba93..1b74108b2 100644 --- a/public/javascripts/aspect-edit.js +++ b/public/javascripts/aspect-edit.js @@ -3,18 +3,6 @@ * the COPYRIGHT file. */ - -$('#move_friends_link').live( 'click', function(){ - $.post('/aspects/move_friends', - { 'moves' : $('#aspect_list').data() }, - function(){ $('#aspect_title').html("Groups edited successfully!");}); - - $(".person").css('background-color','none'); - $('#aspect_list').removeData(); - $(".person").attr('from_aspect_id', function(){return $(this).parent().attr('id')}) - -}); - function decrementRequestsCounter(){ var old_request_count = $(".new_requests").html().match(/\d+/); @@ -30,6 +18,19 @@ function decrementRequestsCounter(){ } $(function() { + + + $('#move_friends_link').live( 'click', function(){ + $.post('/aspects/move_friends', + { 'moves' : $('#aspect_list').data() }, + function(){ $('#aspect_title').html("Groups edited successfully!");}); + + $(".person").css('background-color','none'); + $('#aspect_list').removeData(); + $(".person").attr('from_aspect_id', function(){return $(this).parent().attr('id')}) + + }); + $("ul .person").draggable({ revert: true });