Получи сначала предыдущий активный элемент, если найден, то убери у него класс active.
Вместо этого уродства
if ( content.style .display === "block") {
content.style .display = "none";
} else {
content.style .display = "block";
}
сделай все на css, для этого селектор "+" используется.
И обработчик клика на все элементы накидывать необязательно. Это делается на самый верхний элемент и в самом обработчике через таргет получается текущий. Почитай, что такое всплытие событий.
вот скрипт:
<script>
var coll = document.getElementsByClassName("collapsible");
var i;
for (i = 0; i < coll.length; i++) {
coll[i].addEventListener("click", function () {
this.classList.toggle("active");
var content = this.nextElementSibling;
if ( content.style .display === "block") {
content.style .display = "none";
} else {
content.style .display = "block";
}
});
}
</script>