XaraX: Class mit JS wechseln

Beitrag lesen

Hallo Jürgen,

<style type="text/css">

/* ist valide */

</style>

<script language="JavaScript"> {

^^^
//SyntaxError! Die schließende Klammer fehlt. Abgesehen davon ist sie überflüssig.

// Bei Link auf aktuelle Seite die CSS Klasse auf aktiv setzen
function highlightCurrentPageLink() {
// Alle Links durchlaufen und pruefen
var anzHrefs = document.getElementsByTagName("a").length;
for (var i = 0; i <= anzHrefs - 1; i++) {
// Link auf aktuelle Seite gefunden, also umsetzen CSS auf aktiv
if (document.getElementsByTagName("a")[i].href == document.location.href) {
var currentClassName = document.getElementsByTagName("a")[i].className;
var newClassName = currentClassName.replace(/inaktiv/,"aktiv");
document.getElementsByTagName("a")[i].className = newClassName;

// Es ist in der Tat mühsig sich hier durchzuarbeiten

}
}
}

</script>

//-->

^^^^^
Willst Du dies im <head> wirklich haben?

function highlightCurrentPageLink()
        {
        // Dem Code fehlt es an Abstraktionen, die ihn ver-
        // einfachen.
        // so simpel sieht mein Code aus und er arbeitet!
        // Übersichtlichkeit ist alles!
        var x = document.getElementsByTagName("a");
        for (i=0;x[i];i++)
                {
  // Hier gibts Du Dir bitte ersteinmal per
                // alert(document.location.href), ob dieser
                // Fall jemals eintreten wird!
                if(x[i].href == document.location.href)
                        x[i].className='aktiv'
                }
        }

Desweiteren frage ich mich, warum Du die CSS-Klassen "meinecssklasseinaktiv" nennst, aber im JS "aktiv" heißen. Auch das wäre eine mögliche Fehlerquelle

Gruß aus Berlin!
eddi

--
Manchmal trifft es einen doch ganz unverhofft t86591:
> '..."Vorläufig abgebrochen" ist ungefähr so sinnvoll formuliert, wie "einstweilig erschossen" oder "temporär verbrannt"...'
Ich danke Sven für diese Erkenntnis - Gott, was habe ich gelacht ;)