RyanAg06 / MyStuff

Recopilacion de Programas Utilies y Herramientas para Distintos Sistemas.
https://mystuff-ag.vercel.app
1 stars 0 forks source link

revisión de cerrar y abrir sub menús #7

Closed Gio-mustard closed 4 months ago

Gio-mustard commented 4 months ago

Los algoritmos para abrir y cerrar los sub menús potencialmente podrían estar ejecutando código redundante y podríamos corregir esto para optimizar un poco mas su funcionamiento.

Explicación

En el script funciones.js en el evento resize de la ventana se ejecutan 2 funciones.

window.addEventListener("resize", ()=>
{
    abrirSubMenus();
    cerrarSubmenus();
});

estas funciones en conjunto iteran 3 veces en los elementos del menú de navegación y les agregan eventos para abrir o cerrar sub menús.

problematica

El problema es que si el código ya se ejecuto una vez no hay necesidad de hacerlo 2 veces, incluso esto podría ejecutarse decenas de veces mientras se hace un resize a la ventana, esto hará que se ejecute código innecesario .

posible solución

Se podría usar una variable global que funcione como verificador para saber si ya se ejecuto o no el algoritmo.

RyanAg06 commented 4 months ago

esto lo hice ya que al momento de cambiar el tamaño de la ventana los submenus se quedaban abiertos y cuamdo pasabas a modo escritorio todo se quedaba visible, pero puedo verificarlo de nuevo;)

Gio-mustard commented 4 months ago

Podríamos separar las funciones para que los eventos se asignen una vez y que haga la verificación de la resolución siempre para saber si debe o no cerrar el menú lateral.

Gio-mustard commented 4 months ago

Oyeee, recuerda hacer la referencia del commit cuando cierres un issue para poder leer el codigo, tambien puedes poner un comentario aqui con el ID del commit para tener el seguimiento mas facil