for l oop никогда не запускается
document.querySelector
возвращает первый элемент, который удовлетворяет заданному запросу, если его нет, я считаю, что он либо нулевой, либо неопределенный
document.querySelectorAll
возвращает список узлов элементов, который удовлетворяет запросу, если в пустом списке узлов нет элементов
Я предполагаю, что вы хотите создать каждую ссылку, а не только первую, в этом случае ссылки под нав? Если это так, вы можете просто добавить класс к навигации в зависимости от прокрутки.
const nav = document.getElementById('site-navigation');
window.onscroll = function () {
const navHasClass = nav.classList.contains("over-hundred");
if (window.pageYOffset > 100) {
if (!navHasClass) nav.classList.add("over-hundred");
} else if (navHasClass) {
nav.classList.remove("over-hundred");
}
}
и в вашем css
#site-navigation {
background: #406681;
}
#site-navigation .menu-link {
color: #fff;
}
#site-navigation.over-hundred {
background: transparent;
}
#site-navigation.over-hundred .menu-link{
color: blue;
}