phyton

Hallo!
Ich habe folgende Frage: Kann man mit Javascript an alle Links (a) ein rel="lightbox[roadtrip]" hängen? Zum Beispiel so:

vorher:
<a href="bild.jpg"><img src="bild_thumb.jpg"/></a>
<a href="bild2.jpg"><img src="bild2_thumb.jpg"/></a>
<a href="bild3.jpg"><img src="bild3_thumb.jpg"/></a>

nachher:
<a href="bild.jpg" rel="lightbox[roadtrip]"><img src="bild_thumb.jpg"/></a>
<a href="bild2.jpg" rel="lightbox[roadtrip]"><img src="bild2_thumb.jpg"/></a>
<a href="bild3.jpg" rel="lightbox[roadtrip]"><img src="bild3_thumb.jpg"/></a>


danke euch

phyton

Mister Ad

agrajag

Sicherlich geht das.
Einfach alle a-Elemente auswählen und das Attribut anhängen. Mit einem Framework wie JQuery ist das ein Einzeiler:
Ansonsten musst du dir eben alle A-elemente auswählen ( document.getElementsByTagName ) und in einer Schleife das Attribut hinzufügen ( .setAttribute() ).

__________________
Today you...Tomorrow me.

phyton

danke dir! ich hab jetzt mal ein skript geschrieben, aber es funktioniert nicht. kann mir jemand helfen? ich muss diese ganze prozedur in meinem skript machen, weil ich will ja nur bei bild-verlinkungen das rel-attribut haben. schaut es euch an:


<script type="text/javascript">
var links = document.getElementsByTagName("a");
var anzahl = links.length;

for(i=1; i<=anzahl; i++)
{
var endung = links.split(".");
if(endung=="jpg")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="gif")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="png")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="jpeg")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="JPG")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="GIF")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="PNG")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
else if(endung=="JPEG")
{
document.getElementsByTagName("a")[i].setAttribute("rel", "lightbox[roadtrip]");
}
}
</script>

Manko10

Das muss < und nicht <= heißen.
Außerdem würde ich ein switch statt if/else if/else if/else if... benutzen.

Wenn es dann nicht funktioniert, lasse dir die Variablen mal ausgeben.

Ach, und die Code-Tags dürfen nicht nur Moderatoren und anderen Übermenschen benutzen.

__________________
Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”

phyton

hmm klappt immer noch nicht... ich hab mir mal die variablen ausgeben lassen:

links = [object HTMLCollection]
anzahl = 0;

und wenn ich versuche mir was in der for schleife ausgeben zu lassen, kommt gar nichts raus...



phyton

nikosch

Bitte [php] bbCode verwenden Bzw. [code]
ab es funktioniert nicht ist keine sinnvolle Fehlerbeschreibung

__________________
--
One pixel is still too big. Please make it smaller. ASAP.

Initiative Mittelstand.
Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers.
--

Manko10

Ja, das ist klar. Die wird bei 0 ja auch nicht durchlaufen.
Hast du derzeit überhaupt Links auf deiner Seite?

__________________
Refining Linux Advent Calendar series “24 Outstanding ZSH Gems”

phyton

keine angst da gibts genügend und zwar genau 75. hab mein skript angepasst aber es bewirkt nichts:

agrajag

Hi,
wahrscheinlich hast du das script im Head-Bereich stehen - dort wird das JS ausgeführt obwohl das DOM noch nicht komplett geladen ist. Also entweder den Code erst nach dem entsprechenden Event starten (z.B. onLoad) oder den <script>-Teil unter alle Links setzen.


Noch zwei Anmerkungen:
Ich werde nie verstehen, warum Leute sich freiwillig mit purem JS rumschlagen und 1000end Browserbugs selbst umgehen statte einfach JQuery oder andere 1A-Frameworks zu nutzen.
Deinen Code könnte man auch noch etwas entschlacken - ein switch oder noch schöner: ein Array mit den gewünschten Dateiendungen statt tausende Abfragen und tausendfach wiederholtem Code

__________________
Today you...Tomorrow me.

nikosch

Stimmt, das Ganze in einer Funktion zu kapseln wäre sinnvoll. Trotzdem kann ich nur schwer glauben, dass das Script keinen Fehler wirft.

@agrajag: Merke: Ein Framework schützt vor Arbeit nicht.

__________________
--
One pixel is still too big. Please make it smaller. ASAP.

Initiative Mittelstand.
Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers.
--