Укажите пожалуйста, где ошибка (функция выравнивания блока по вертикали и горизонтали)?

Question

[Константин]

Добрый вечер!!

Написал вот такую функцию по выравниванию по центру модального окна по вертикали и горизонтали:

$(function() {

    alignCenter($(elem))

    $(window).resize(function() {
            alignCenter($(elem))
    })

    function alignCenter(elem) {
        elem.css({
            left: ($(window).width() - elem.width()) / 2 + 'px',
            top: ($(window).height() - elem.height()) / 2 + 'px'
            })
        }

})
    var elem=document.getElementById('pop_up')

При resize четко срабатывает если уберу первые 2 строчки. Но нужно чтобы окно сразу центрировалось, а не только при изменения размера окна. Подскажите, пожалуйста, где тут ошибка? не силен в js :(

Answer 1

[Александр]

у вас elem объявляется после тога как должна была бы отработать функция alignCenter
поднимите в коде var elem=document.getElementById('pop_up')
наверх, перед alignCenter

Comments

[Константин]

не помогло. И , как я уже сказал, при ресайзе работает. Но если оберну в самовызывающуюся, сразу перестает

[Александр]

Константин: объявление переменной внутри функции сейчас?
пототму что в выше приведенном коде локальная переменная объявляется за пределами функции
естественно ее не будет видно

[Александр]

что то типа того

$(function() {
    var elem = $('#pop_up');
    
    alignCenter(elem);

    $(window).resize(function() {
        alignCenter(elem);
    });

    function alignCenter(elem) {
        elem.css({
            left: ($(window).width() - elem.width()) / 2 + 'px',
            top: ($(window).height() - elem.height()) / 2 + 'px'
        });
    }
});

[fsockopen]

а обязательно оборачивать это все в

$(function() {
...............
});

[Александр]

не обязательно
и у вас отладчик есть
если что то не работает, то первым делом смотрите на что ругается и когда задаете вопрос пишите какаую ошибку выдает

[Константин]

Проблему нашел. Проблема была не в объявлении переменной и не в обертке. Файл js большой, и чуть выше ошибка в инструкциях, из за которой, он не обрабатывал эти команды

Answer 2

[Илья Пирожок]

Зачем тут использовать js если есть много css способов?