Mejora general

• Utilice let en lugar de var . Mira esto
• Si solo está accediendo a una variable y no la está cambiando, use const en lugar de let .
• document.querySelectorAll() o document.querySelector() son en algunos casos (como en su caso) mejores que getElementsByClassName o getElementById o getElementsByTagName .
• Al iterar sobre matrices o listas de nodos estáticos, forEach es generalmente una opción mejor (más legible) que un bucle for . ( querySelectorAll devuelve una lista de nodos estáticos. No necesita Array.from , pero si desea usar otros métodos específicos de Array, use Array.from como señaló @Yousaf)
• Utilice la function en el ámbito global y para las propiedades de Object.prototype . Utilice class para constructores de objetos. Use => en cualquier otro lugar. Mira esto.

Así es como lo habría escrito

 document.querySelectorAll('.btn-danger').forEach(btn=>btn.addEventListener('click',e=>e.target.parentElement.parentElement.remove()))

Su código se puede acortar a:

 Array.from(document.getElementsByClassName('btn-danger')).forEach((button) => { button.addEventListener('click', (e) => { e.target.parentElement.parentElement.remove(); }); });

Esto usa:

• Array.from().forEach() :
  • Cómo iterar correctamente a través de getElementsByClassName >
• Funciones de flecha
  • ¿Cuál es el significado de "=>" (una flecha formada por igual y mayor que) en JavaScript?

Dado que las funciones de matriz se pueden reescribir sin {} , una sola línea se vería así:

 Array.from(document.getElementsByClassName('btn-danger')).forEach((button) => button.addEventListener('click', (e) => e.target.parentElement.parentElement.remove()));