Фиксированная шапка сайта, при скроле по id, залезает на секцию со встречным id. как исправить?

Question

[Alex Ivanov]

Доброe времени суток!
Возможно кто сталкивался с такой причиной!?
Делаю лендинг, шапка сайта фиксированная и при нажатии на меню, должна скроллить в низ по id.

Столкнулся с тем, что по нажатию шапка скроллит в низ, но наезжает на встречную секцию(((
Сама шапка 125рх, то есть эти 125рх наезжают на секцию (место прибывания)

Вот основной спритп фиксирования:

var top_show = 125;
    $(document).ready(function() {
        if ($(this).scrollTop() > top_show){
            $('header').css("position","fixed");
        }
        else{
            $('header').css("position","relative");
        }
        $(window).scroll(function () {
            if ($(this).scrollTop() > top_show){
                $('header').css("position","fixed");
            }
            else{
                $('header').css("position","relative");
            }
        });
    });

Вот дописал еще один, чтобы при скроле, было -125рх.
Т.е. чтобы шапке не заежала на секцию

<script>
    $(document).ready(function(){
        $('.menu ul li a').click(function(){
            elementClick = $(this).attr('href');
            destination = $(elementClick).offset().top-125;
            if ($.browser) {
                $('body').animate({scrollTop: destination}, 1000);
            }
            else {
                $('html').animate({scrollTop: destination}, 1000);
            }
            return false;
        });
    });
</script>

Но почему то не всегда срабатывает и периодически пишет ошибку на

destination = $(elementClick).offset().top-125; //на сам .top

Подскажите, пожалуйста, может есть другое решение???

Comments

[Alex Ivanov]

https://semenovalex.com/

Вот ссылка.
Сейчас вообще работает, не понятно как.
Консоль красная(((

[ⓒⓢⓢ]

У тебя расходиться описание с реальным кодом на сайте
https://yadi.sk/i/cV2We2c13UCkhr

Комментарии в коде смотри - поймёшь из-за чего ошибка

$(document).ready(function(){
        $('header .container').click(function(){
            elementClick = $(this).attr('href'); // elementClick = undefined  // (у $(this) нет аттрибута href)
            destination = $(elementClick).offset().top-90; // $(undefined).offset().top-90; // -> Uncaught TypeError
            if ($.browser) {
                $('body').animate({scrollTop: destination}, 1000);
            }
            else {
                $('html').animate({scrollTop: destination}, 1000);
            }
            return false;
        });
    });

spoiler

Answer 1

[be_a_man]

1. Непонятно зачем скриптами фиксить панель, когда это делается c помощью CSS.
position: fixed на панель,
а на body padding-top: 125px

2.

$('.menu ul li a').click(function(){
	var el = $(this).attr('href');
	$('html,body').animate({
		scrollTop: $(el).offset().top - $("header").height()}, 1000);
	return false;
});