Как запустить функцию только один раз?

Question

[Андрей Чернов]

Добрый день.
При скролле страницы функция находит элемент с указанным классом elem и к элементу с id popup-item добавляет новый класс, который делает видимым попап (добавляет display: block).
Если закрыть попап и продолжить скроллить сайт, попап заново открывается.
Как сделать, чтобы функция выполнялась только один раз и не вызывалась повторно?

Полный код https://jsfiddle.net/devchernov/6zc53dkj/20/

Код скролла

jQuery(document).ready(function($) {
           var target = $('.elem');
                var targetPos = target.offset().top;
                var winHeight = $(window).height();
                var scrollToElem = targetPos - winHeight;
                $(window).scroll(function(){
                  var winScrollTop = $(this).scrollTop();
                  if(winScrollTop > scrollToElem){
                        $('#popup-item').addClass('popup-open')
                    }
                  });
                });

Answer 1

[Rsa97]

$('#popup-item').not('.done').addClass('popup-open done');

Comments

[Павел Корнилов]

+1
https://jsfiddle.net/py39qtdg/

Answer 2

[0xD34F]

Удаляйте обработчик после того, как он выполнил нужное действие:

$(window).scroll(function onScroll() {
  const $this = $(this);
  if ($this.scrollTop() > scrollToElem) {
    $('#popup-item').addClass('popup-open');
    $this.off('scroll', onScroll);
  }
});

Answer 3

[Максим Тимофеев]

Добавить флаг
var popup_opened = false
в коде проверять если он false выполнять код и менять его на true, если там true, то соответственно ничего не делать.
Если надо решать вопрос и при повторных загрузках страницы - то писать флаг в localStorage