search

midudev / covid-vacuna

App para ver el estado de la vacunación COVID-19 en España
https://covid-vacuna.app
Apache License 2.0
365 stars 126 forks source link

Modifica alineación de progressBar al centro #11

Closed YoelRibero closed 3 years ago

YoelRibero commented 3 years ago

Tal vez la alinación que tiene actualmente te gusta pero me parece una buena idea para romper el hielo, saludos!! 😀

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/midudev/covid-vacuna/6fuakqn1p
✅ Preview: https://covid-vacuna-git-fork-yoelribero-issue-align-progress.midudev.vercel.app

midudev commented 3 years ago

Me sale con tus cambios un pelín desalineada. Eso es porque el contenedor ese ocupa más que la barra de progreso... habría que hacer otros cambios para que quedase exactamente en el centro con ese CSS.

Por eso he usado el position: absolute. No está bien... pero no tenía tiempo de hacer el resto de cambios. XD

YoelRibero commented 3 years ago

Excelente, muchas gracias por el feed back, me pasa que al ponerle el flex al progress mismo (no al padre), el padre pasa a ocupar el alto del contenido y no como antes, incluso ahora me estaba dando cuenta que ni siquiera hace falta alinearlo al centro con el flex bastaría, también es importante quitarle el top al ::before, si quieres y puedes pruébalo y me dices

midudev commented 3 years ago

Me la revisaré para refactorizar para poder usar flex! Gracias crack!