Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Agregar modules a todos los SCSS #63

Open
anaggithub opened this issue Sep 29, 2021 · 0 comments
Open

Agregar modules a todos los SCSS #63

anaggithub opened this issue Sep 29, 2021 · 0 comments

Comments

@anaggithub
Copy link
Collaborator

anaggithub commented Sep 29, 2021

Actualmente nuestros archivos scss no son modulos. Esto quiere decir que si en un componente colocamos como ejemplo una clase de nombre "container", y luego en otro componente colocamos una clase del mismo nombre, a nuestro componente le cargará los estilos de ambas clases, ya que nuestros archivos de estilos no son módulos independientes.

Para resolver esto necesitamos agregar ".modules" en los nombres de los archivos scss, los cuales quedarían así: styles.modules.scss.
image
Y obvio arreglar el import de dicho estilo:
image

Hacer esto archivo por archivo y correr la app para revisar que no se rompan los estilos. Algunos estilos podrían verse rotos, ya que hemos estado usando nombres de clases muy muy parecidas y clases iguales en varios componentes (ejemplo: "cards-container").

Si se rompe algo muy muy complejo de arreglar, avisar y así vemos si creamos un ticket diferente para arreglar esos estilos en particular.

Es importante arreglar esto porque es muy mala práctica que los estilos de un componente se carguen en otros componentes ajenos y estos deberían estar encapsulados y privados a su componente.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant