4
respostas

Pequena correção

Nas linhas de comando abaixo estamos atrelando o evento a toda a tag table, porém se clicarmos ele aceita remover até o cabeçalho da tabela toda, esse problema foi corrigido com a solução de trocarmos var tabela = document.querySelector("table"); por var tabela = document.querySelector("tbody");, com o tbody estamos anexando o evento somente ao copo da tabela e não a tabela toda em si.

4 respostas

Boa Victor! Ótima observação, parabéns :D

Valeu Victor! Com essa alteração o remove-paciente agora funciona com o console aberto também. O double click não estava funcionando no meu Chrome com o console aberto. Fui em settings do console e lá embaixo cliquei em 'Restore defaults and reload', após isso voltou a funcionar normalmente.

Assim também, funciona:

var tabela = document.querySelector("#tabela-pacientes");

É até uma boa pratica usar os ids ao invés das tags.

Concordo com você, ótima colocação mas talvez em um caso de manutenção o dev poderia mudar esse id e sem saber que ele impactaria em outros lugares. Uma outra opção seria usar data attributes. Mas parabéns pelo reforço no tópico. Abraços