hallo
Hallo beatovich,
Viel weniger Code
Meine Meinung dazu: Das ist jetzt zu wenig. Parameter zu verwenden, um sich let oder var zu sparen, ist schlechter Stil. Das darf kann ein Mini-/Uglifizierer ausspucken, aber ein Mensch sollte das nicht tun. Und Variablen sollte man sprechend benennen.
Aber wo wir gerade dabei sind - könnte man das Folgende irgendwie zum Funktionieren bringen? Wenn man es einfach so verwendet wie geschrieben, könnte - je nach Seitenadresse - ein ungültiger Selektor herauskommen. Wie muss man escapen?
function setAriaCurrentPage(){ var selfNodes = document.querySelectorAll("a[href"+location.href+"]");
Du glaubst doch nicht etwa, dass href Attribute volle URIs enthalten.
for(i=0; i<selfNodes.length; i++){ selfNodes[i].setAttribute("aria-current","page"); } }