Tengo la siguiente declaración if else:
let userIconText = document.getElementsByClassName("iconText"); let userIconDiv = document.getElementById("userIcons"); let rCorners2 = document.getElementById("rcorners2"); let homeContainerDiv = document.getElementById("homePanel"); let icons = document.querySelectorAll('#userIcons li'); document.getElementById("menuFilter").onclick = function(){ for (let i = 0; i < userIconText.length; i++) { for (let j = 0; j < icons.length; j++) { if (userIconText[i].style.display !== 'block') { userIconText[i].style.display = "block"; console.log("1"); userIconDiv.setAttribute("style","width:10vw"); icons[i].setAttribute("style","margin:75px 20px 0 10px"); console.log("2"); homeContainerDiv.setAttribute("style","width:85vw; transform: translateX(5%);"); rCorners2.setAttribute("style","width: 53vw; left: 295px"); console.log("3"); } else { userIconText[i].style.display = "none"; console.log("4"); userIconDiv.setAttribute("style","width:5vw"); homeContainerDiv.setAttribute("style","width:90vw"); console.log("5"); icons[i].removeAttribute("style", "margin"); rCorners2.removeAttribute("style", "width"); console.log("6"); } } } }
Ahora esto estaba funcionando, pero ahora, por alguna razón, todo en esta declaración se está ejecutando, por lo tanto, tengo el registro de la consola de los números para probar esto, cuando hago clic en "menuFilter", reviso el registro de la consola y muestra los números. Del 1 al 6. Simplemente no entiendo por qué sucede esto ahora, funcionó ayer y ahora tengo este problema. Entonces, realmente, cuando hago clic por primera vez en "menuFilter", solo debería ser todo antes de que se ejecute el otro y luego, cuando hago clic nuevamente, debería ser todo después del otro. Cualquier consejo sobre cómo ordenar esto sería muy apreciado.
El if-else está dentro de dos bucles for.
En diferentes repeticiones, puede desencadenar diferentes ramas.
Eso es lo que puedo decir según el código que publicaste.
Resolví el problema, gracias por la ayuda. Para el userIconText hay 7 elementos y debido a un cambio que se está realizando, en los íconos hay 8 elementos, por lo que parece que debido a eso, se estaba ejecutando una repetición adicional de la que debería haber sido lo que estaba causando que se ejecutaran las partes después. la otra condición también.
Entonces, lo que he hecho es agregar lo siguiente al segundo ciclo for:
for (let j = 0; j < icons.length; j++) { if(icons[j].classList.contains("hidden-search")) { continue; }