jQuery TR plus proche sélection

Question

Espérons que quelqu'un peut vous conseiller. Avoir des problèmes en essayant de supprimer une ligne une fois qu'un lien a été cliqué.

HTML

<table> 
    <tr><td>Some Data</td><td><a href="#" class="remove-row>Remove Row</a></td></tr> 
    <tr><td>Some Data</td><td><a href="#" class="remove-row">Remove Row</a></td></tr> 
</table> 

Maintenant, la JS

$("a.remove-row").live('click', function(eve){ 
    eve.preventDefault(); 
    $.ajax({ 
    type: 'GET', 
    url: '/someaction/', 
    dataType: 'json', 
    success: function(msg){ 
    if(msg.error){ 
    alert(msg.error); 
    }else{ 
    $(this).closest('tr').remove(); 
    alert(msg.success); 
    }  
    } 
    }) 
}); 

Cela devrait être simple, réel BUTS son ne pas enlever la ligne. Juste pour coups de pied si je le change à quelque chose comme

$('.remove-row').addClass('foo'); 

Il va ajouter toto à toutes les lignes de la table. Donc, peut comprendre pourquoi il ne supprime pas la ligne la plus proche.

Des idées?

Merci à l'avance.

Answer 1

Le problème est this fait actuellement l'objet ajax dans votre rappel success, mais il est une solution facile, utilisez l'option content comme ceci:

$("a.remove-row").live('click', function(eve){ 
    eve.preventDefault(); 
    $.ajax({ 
    context: this,     //add this here! 
    type: 'GET', 
    url: '/someaction/', 
    dataType: 'json', 
    success: function(msg){ 
    if(msg.error){ 
    alert(msg.error); 
    }else{ 
    $(this).closest('tr').remove(); 
    alert(msg.success); 
    }  
    } 
    }) 
}); 

L'option context dicte ce que this sera dans le $.ajax() fonctions de rappel, puisque vous voulez que ce soit le .remove-row sur lequel vous avez cliqué, utilisez this comme option.

Answer 2

Vous avez l'attribut non fermé class="remove-row à la première ligne.

See here

Answer 3

réponse de Nick devrait fonctionner, mais vous pouvez aussi le faire, je ne sais pas quel est le meilleur, probablement son Nick, mais il peut aider quand même ...

$("a.remove-row").live('click', function(eve){ 
    var row = this; 
    eve.preventDefault(); 
    $.ajax({ 
    type: 'GET', 
    url: '/someaction/', 
    dataType: 'json', 
    success: function(msg){ 
    if(msg.error){ 
    alert(msg.error); 
    }else{ 
    $(row).closest('tr').remove(); 
    alert(msg.success); 
    }  
    } 
    }) 
}); 

Answer 4

Ne serait-il pas plus facile de faire la suppression/cacher avant la main?

comme ceci:

$("a.remove-row").live('click', function(eve){ 
     $(this).hide(); 
     <The rest of your code logic> 
     ......