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? CSS.escape() ist ja noch nicht wirklich Standard, gibt's dafür einen brauchbaren Polyfill?
function setAriaCurrentPage(){
var selfNodes = document.querySelectorAll("a[href"+location.href+"]");
for(i=0; i<selfNodes.length; i++){
selfNodes[i].setAttribute("aria-current","page");
}
}
Rolf
--
sumpsi - posui - clusi
sumpsi - posui - clusi