0

Div einblenden und zum Anker scrollen

Started by novoo, October 22 2011 12:05 PM

You cannot reply to this topic

4 replies to this topic

#1 novoo

Forumsäule

~Mitglieder
3,052 posts

Posted 22 October 2011 - 12:05 PM

Moin,

ich sitz hier gerade an nem Wordpress Template, bei dem die Kommentare ausgeblendet sind und via Klick eingeblendet werden können, das funktioniert auch.

Jetzt will ich aber, dass beim Klick auf "Einblenden" noch gleichzeitig zum Anker "kommentare" gescrollt werden soll. Wie löse ich das jetzt?

Vielleicht ist mein ganzer Code auch totaler Crap und jemand (Flo

) hat ne bessere Lösung.

Hier mal der Code: http://jsfiddle.net/UdRd6/

/edit

habs jetzt mit

<a id="displayText" onclick="javascript:toggle();" href="#kommentare">Kommentare anzeigen</a>

gelöst. Gibt's da aber nicht trotzdem ne schönere Lösung?

Edited by novoo, 22 October 2011 - 01:02 PM.

trstn.de

Back to top

#2 Pichfl

Refactoring ftw!

Admins
2,752 posts

Name: Florian Pichler

Posted 22 October 2011 - 01:21 PM

1. Javascript über javascript: im href auszuführen ist mehr als unschön. Der richtige Weg sind hier Eventhandler oder wenn es nicht anders geht die klassischen onxxxx-Attribute im HTML.
2. display:none per CSS zu setzen bedeutet, dass man die Kommentare ohne JavaScript niemals lesen kann. Besser display erst mit JS zu setzen. Noch besser: Überhaupt kein CSS per JS zu setzen und stattdessen nur Klassen austauschen

Wenn du Wordpress hast, wirst du sowieso jQuery eingebunden haben, also ist es eine Überlegung wert, das ganze über jQuery zu lösen.

Lösung bei JSFiddle: http://jsfiddle.net/JCDA8/17/

Das Scrollen ist in meinem Code eigentlich eher ein Nebeneffekt, bzw der Tatsache geschuldet, dass ich die normale Reaktion des Browsers auf einen Link mit # nicht abschalte (wie das geht steht aber kommentiert im Code).
Du könntest es deaktivieren und stattdessen auf irgendein SmoothScrolling-Plugin für jQuery zugreifen, das erste was ich gefunden hab ist https://github.com/k...y-smooth-scroll

(jQuery ist nicht gerade mein Hauptgebiet)

Florian Pichler