Keine Ahnung von Javascript --

Hallo ihrs,

ich hab mir hier einen Schnipsel besorgt der die Navi oben (.nacont) fest pinnt
jetzt habe ich aber auf einer anderen Unterseite ebenfalls eine Navi (.buchen) die gepinnt werden soll. jedoch ist diese anders gestylt.

Hier mein Script , vielleicht jemand da der mir sagen könnte warum das nicht geht ?

Code:

---

<script>
    $(document).ready(function() {
                        var top = $('.nacont').offset().top;
                        $(window).scroll(function() {
                        if($(this).scrollTop() > top) {
                                $('.nacont').addClass('fixed');
                        } else {
                                $('.nacont').removeClass('fixed');
                        }
                });
        });
       
        $(document).ready(function() {
                        var topa = $('.buchen').offset().top;
                        $(window).scroll(function() {
                        if($(this).scrollTop() > topa) {
                                $('.buchen').addClass('fixed');
                        } else {
                                $('.buchen').removeClass('fixed');
                        }
                });
        });
</script>

---

Probier mal

Code:

---

$(document).ready(function() {

    var top = $('.nacont').offset().top;
    $(window).scroll(function() {
        if($(this).scrollTop() > top) {
            $('.nacont').addClass('fixed');
        } else {
            $('.nacont').removeClass('fixed');
        }
    });

    var topa = $('.buchen').offset().top;
    $(window).scroll(function() {
        if($(this).scrollTop() > topa) {
            $('.buchen').addClass('fixed');
        } else {
            $('.buchen').removeClass('fixed');
        }
    });
});

---

also alles in einem $(document).ready(...) Block.

Alternativ könntest du deinen verschiedenen Navigationen auch eine zusätzliche Klasse .pinNavi verpassen und dann in deinem ursprünglichen Script (das mit nur einer Navigation) .nacont durch .pinNavi ersetzen.
Vorteil: wenn du mal noch eine andere Navigation nutzen willst, dann musst du das Script nicht mehr anfassen und nur wieder die Klasse .pinNavi vergeben.

Gruß, folkfreund

Warum geht das nicht mit der Version von fiedsch , klärt mich bitte mal auf will ja nicht dumm sterben.

Danke für eure Antworten !

@fiedsch : Die Version hat leider auch nicht funktioniert.

@folkfreund : Jo die Vorteile liegen auf der Hand, ich hab das jetzt auch so gemacht

Zitat:

---

Zitat von Membaris

Warum geht das nicht mit der Version von fiedsch , klärt mich bitte mal auf will ja nicht dumm sterben.

---

Wahrscheinlich, weil mein Vorschlag zwar den Code eleganter macht, aber nichts mit dem eigentlichen Problem zu tun hat. Soweit meine kurze Recherche korrekt ist, kann man durchaus mehrere
$(document).ready(...) Blöcke haben. Bei $(window).scroll(...) sollte es genauso sein.

Was das eigentliche Problem ist -- Javascript ist nicht so sehr meins -- kein Ahnung.

Du ich wollte dich nicht kritisieren, es würde mich nur mal interessieren wie es aussehen muss das es klappt.

Gesendet von meinem GT-I9505 mit Tapatalk