midudev / la-velada-web-oficial

Web oficial de La Velada IV de Ibai Llanos
https://lavelada.es
Other
1.37k stars 621 forks source link

feat: refactor DOM selectors, using `@/lib/dom-selector` #934

Closed AlejandroSuero closed 4 months ago

AlejandroSuero commented 4 months ago

Descripción

Refactorización haciendo uso de @/lib/dom-selector.

Problema solucionado

Aprovechar el uso de los selectores que tenemos.

Cambios propuestos

  1. document.getElementById(...) -> $(#...)
  2. document.querySelector(...) -> $(...)
  3. document.querySelectorAll(...) ->$$(...)`

Capturas de pantalla (si corresponde)

Comprobación de cambios

Impacto potencial

Mejorar la consistencia del uso de los selectores.

Contexto adicional

Enlaces útiles

vercel[bot] commented 4 months ago

@AlejandroSuero is attempting to deploy a commit to the midudev pro Team on Vercel.

A member of the Team first needs to authorize it.

pjmartorell commented 4 months ago

@AlejandroSuero las paginas que contienen el import { $ } from "@/lib/dom-selector" con <script is:inline> me dan el siguiente error en local y en producción y funcionalidades como Agregar al calendario dejan de funcionar. Al usar querySelector()no hay ningún problema.

Captura de pantalla 2024-05-06 a les 21 49 20