Вы можете просто добавить прослушиватель событий в свой контейнер изображений. Не нужно добавлять css и не нужно передавать функцию из вашего div. В вашем скрипте тега это то, что вы можете сделать ... Таким образом, ваш JavaScript управляет шаблоном, а не смесью вещей.
// for getting the button
let button = document.querySelector('button');
// hiding the image div at first
document.getElementById('flyingcat').style.display = 'none';
// event listner on click
button.addEventListener('click', () => {
// getting the image
const image = document.getElementById('flyingcat');
// toggle image display
if (image.style.display === 'none') { // if the diplay is none
image.style.display = 'block'; // making the image block
} else image.style.display = 'none'; // else hiding the image again
});
Я прокомментировал строки, чтобы вы знали, что делается в какой момент ...
Я вижу, вы использовали видимость: скрыто. Позвольте мне сказать вам, что при использовании этого элемента он был бы там, но его видимость была бы ничем, означая, что он будет всегда покрывать пространство. Но используя свойство display: none, мы можем удалить весь элемент с экрана.