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

Fixs styles errors #2

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fixs styles errors #2

wants to merge 4 commits into from

Conversation

oyham
Copy link

@oyham oyham commented Dec 29, 2023

EN

Correcciones de estilos y pequeñas mejoras. ( revisar el archivo notes.txt, posible error en msg error de la response al no poder iniciar sesión ) (username is invalid -> email is invalid)

Las implementaciones y cambios que realicé fueron:

  • solucionar el error en enp, dónde quedaba el loader en loop
  • corrección en issues, achicando el header left y añadiendo overflow-y a filters
  • redefiniendo preferences quitando username img en personal details y añadir overflow
  • achicar section left de web para una mejor separación entre las dos secciones principales.

Revisar la carpeta screenshots para ver el antes y después.


ES

The implementations and changes I made include:

  • resolving the error in enp where the loader was stuck in a loop.
  • correction in issues, reducing the left header size and adding overflow-y to filters.
  • redefining preferences by removing the username image in personal details and adding overflow.
  • reducing the left section of the web for better separation between the two main sections.

Please review the screenshots folder to see the before and after.

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

Successfully merging this pull request may close these issues.

1 participant