Есть ли причина, по которой вы используете куки? Если нет, вы можете использовать localStorage и написать что-нибудь немного чище ..
Так например
$(document).ready(function() {
// if localStorage doesnt have the shownPopUp item
if(!localStorage.getItem('shownPopUp')){
$('.popup-wrapper').css('display','block');
}
$('.popup-close').click(function(){
// set the shownPopUp item in localStorage
localStorage.setItem('shownPopUp', true)
$('.popup-wrapper').css('display','none');
});
});
});
Нет необходимости в функции getCookie
Cookies больше используются для серверных функций, где localStorage лучше для клиентских функций. Так что здесь произойдет, ваши пользователи никогда не увидят всплывающее окно, если они не удалили там localStorage