Почему перестает работать страница при закрытии всплывающего окна?

Question

[Nik Master]

Нужно было скрыть один попап при открытии другого. Написал код, но после закрытия уже второго окна перестает работать кнопка и прокрутка на странице до перезагрузки. Подскажите что дописать или подправить? Только не нужно посылать на изучение документации, сейчас нет времени, лучше не отвечайте если нет другого решения.) Ссылка на сайт. Нажимаем на Обратную связь, потом Отправить email.

$(document).ready(function(){
  $("#hider").click(function () {
    if ($("#pum-235").show()) {
      $(".mfp-wrap").hide() + $(".mfp-bg").hide();
    }
  });
});

Comments

[goshaLoonny]

Документацию читать нет времени, а ждать ответ часами время есть? Нет времени и нет желания это разные вещи

[Nik Master]

goshaLoonny, Ты если не знаешь так не пиши. У меня почти на все вопросы ответы даны мной же. Я ищу информацию сам. Когда уже нет вариантов пишу вопрос и продолжаю дальше искать.

[goshaLoonny]

Nik Master, только что вы писали "Только не нужно посылать на изучение документации, сейчас нет времени", а теперь пишете "продолжаю искать дальше". Ну да, если искать дальше минуя документацию так явно быстрее выйдет, удачных поисков, что могу еще вам пожелать. Надеюсь увидеть от вас статью на хабре "Гайд: Как работать контрпродуктивно"

Решение кстати очень простое и для этого не нужна документация, а всего лишь капля здравого смысла и логики.

[Nik Master]

goshaLoonny, Так а смысл писанины вашей? Нет помощи так мимо. Или свербит в одном месте.)

[goshaLoonny]

Nik Master, мда.. Ну если дать верный ответ несмотря на хамское поведение это не помощь, то что тогда? Надеюсь хотябы правильным решением отметить вам гордыня не помешат)

[goshaLoonny]

Nik Master, Кстати вам бы ответили намного быстрее если бы вы не отметили вопрос как "Сложный"

[Nik Master]

goshaLoonny, А где простите хамское поведение? Просто сказал что писать не нужно если нет ответа. А по документации так написано потому что кидали уже ссылку на Magnific Popup и там я явно быстро ответ не соберу так как знания в js поверхностные.)

[goshaLoonny]

Nik Master, А зачем вас туда посылали если вы кастомный popup сделали? Очень странные ребята у вас в советчиках

[Nik Master]

goshaLoonny, Ребята говорят костыли горожу. По другому нужно делать.

[goshaLoonny]

Nik Master, ну по факту говорят. Если уж и использовать библиотеку, то везде. Но на одностраничнике подключать целую библиотеку для пары папов это немного странно. Лучше б вы везде свои костыли использовали. Хотя по факту костыль бы использовался везде один и тот же

[Nik Master]

goshaLoonny, Тоже так думаю. К тому же код чужой, не с ноля писал.

[goshaLoonny]

Nik Master, А точки возле кнопок это какое-то стилистическое извращение? Или list-style-type: none использовать по религии не положено?)

[Nik Master]

goshaLoonny, Это тестовый, на боевом всё ровно.)

[goshaLoonny]

Nik Master, Кстати в magnificPopup есть функция для закрытия текущего открытого попапа, я не вдавался что вы тут его используете просто
$.magnificPopup.instance.close()

[Nik Master]

goshaLoonny, Буду иметь ввиду.

[goshaLoonny]

Nik Master, А открыть новый попап можно вот так

$('button').magnificPopup({
  items: {
      src: '#popup', // Это элемент контейнер в который будет в открытом попапе, так же можно добавить сюда HTML код
      type: 'inline'
  }
});

Answer 1

[goshaLoonny]

Конструкция $("#pum-235").show() всегда будет true в контексте if else, так как возвращает объект. Следовательно зачем она нужна в условии не совсем ясно.
Вот тут вы уже влезаете в чужой код: $(".mfp-wrap").hide() + $(".mfp-bg").hide();Влезать в чужой код, а потом удивляться что он перестал работать глупо, вам так не кажется? Если вам нужно закрыть тот popup, то закройте его, нет необходимости нарушать работу чужого кода. Есть кнопка которая его закрывает, вот она:

<button title="Close (Esc)" type="button" class="mfp-close">×</button>

эмулируйте нажатие этой кнопки - $('.mfp-close').click() и никакой проблемы не будет

$('#hider').click(() => {
  $('.mfp-close').click() // Скрыли предыдущий popup 
  $('#pum-235').show() // Показали ваш
});

Comments

[goshaLoonny]

Кстати обратите внимание еще вот на это

[Nik Master]

goshaLoonny, Это что, не совсем понял?

[goshaLoonny]

Nik Master, Это у вас кнопка окрашивается после того как на нее нажал. Это все же ссылка

[Nik Master]

goshaLoonny, Спс. Посмотрю.