open-source-uc / old-web-osuc

Este repositorio contiene el sitio web del proyecto open source de la UC.
https://osuc.dev
MIT License
3 stars 1 forks source link

Mobile nav fix #65

Closed a-maccormack closed 1 year ago

a-maccormack commented 2 years ago

1. Los vinculos de la barra de navegación móvil aparecen por debajo de la sección "hero"

En la vista móvil del sitio actual [768px, 576px, 320px], los links del menú colapsable aparecen encima de la sección "hero".

Actualmente:

before

Fix Propuesto [Fix 1]:

after

Solución: Aumentar el z index y poner un color de fondo para tapar la sección "hero" cuando el menu esté abierto

Pasos tomados en src/css/stisla.css:

  1. Usé un z-index similar al máximo de 999999 que encontré en el code base.
  2. Agregué !important para arreglar la posición
  3. Agregué un color de fondo a .navbar-expand-lg de valor #000 (color utilizado en el theme del sitio)

2. Subtitulo lead aparece debajo de nav EN CHROME

En la vista móvil del sitio actual [solo en 768px], la sección "Estudiantes apasionados por el código abierto" (p class="lead"), aparece por muy cerca de la barra de navegación. Además, al implementar el Fix #1, termina apareciendo por debajo (desaparece).

Actualmente CHROME:

Screenshot from 2022-03-09 19-41-47

Después del [Fix 1]:

Screenshot from 2022-03-09 19-47-38

Fix Propuesto [Fix 2]:

Screenshot from 2022-03-09 19-49-43

Solución: Aumentar margin-top a 100px en el media frame 768px

Pasos tomados en src/css/stisla.css:

  1. margin-top de p.lead 100px

Notas finales:

  1. Ambas soluciones arreglan los problemas descritos en Chrome y Firefox.
  2. Si es que algún metodo utilizado en mi solución, no es el preferido, por favor dejar feedback para poder desarrollar con otro metodo preferido.
  3. No entendí muy bien lo del members-manifest.json. Supongo que está bien como está actualmente (no hice ningún cambio)

Gracias de antemano.