Challenge 31 #2
Challenge 31 #2
Conversation
app.js
Outdated
| }, | ||
|
|
||
| removeRegister: function removeRegister(e) { | ||
| this.parentNode.parentNode.remove(); |
There was a problem hiding this comment.
Fala @gustavom! O problema na verdade está na forma de remover o elemento! Provavelmente o this aqui não é quem você esperava que fosse =)
O ideal, para evitar esse tipo de problema, é tentar evitar ao máximo o uso do this. No caso dessa aplicação, você pode precisar dele em alguns momentos, mas tente usar somente onde você sabe que ele não vai mudar, o que não é o caso de callbacks de evento =)
Então, pra resolver, você pode fazer o seguinte: adicione um identificador único em cada linha.. quando clicar em remover, verifique por esse identificador, e então remova a linha. Deu pra sacar? =)
|
Opa @fdaciuk , beleza? Saquei, essa parada do 'this' parece fácil, mas não é né. |
|
É meio chatinho lidar com |
Adicionei a opção de remover o registro.
Mas não consegui utilizar a lib DOM para isso, fiz com js puro, pois apareceu um bug, o último registo não era removido, ai fiz assim:
$btnRemove.addEventListener('click', this.removeRegister, false);mas dessa maneira não funcionou:
$('[data-js="btn-remove"]').on('click', this.removeRegister);