Sección de patrocinadores en la home#104
Conversation
Also add a TextBox generic component
5e383f3 to
1814fe1
Compare
There was a problem hiding this comment.
Request changes:
En móvil se superponen las letras:
En móvil hay mucho padding, queda el texto muy estrecho en una sola columna:
Los colores de fondo de los tiers se alejan del código de colores corporativos
Los colore que propone el diseñador son colores fuego, pero los títulos aparecen con unos colores frios:
Esta paleta de fuego, equilibrada con el rigor del negro, crea un sistema visual de alto impacto que es a la vez tecnológico
Control TextBox
2026.es.pycon.org/src/components/TextBox.astro
Lines 9 to 13 in ac342f4
Hay un nuevo control <TextBox, pero un cuadro de texto está pensado para editar texto, esto solo lo muestra. Si es un control genérico pensado para dar formato a un párrafo, quizá le podríamos encontrar otro nombre ( CenteredPanel ?) .
Alt text logos
Los logos tienen como alt-text el nombre del patrocinador que se repite luego bajo el logo:
<img src="data:image/svg+xml;base64,PH..." alt="Sponsor 1" width="150">Quizá sería más explícito indicar que ese es el logo del patrocinador:
<img src="data:image/svg+xml;base64,PH..." alt="Logo de Sponsor 1" width="150">Questions
Pregunto sobre decisiones para entenderlas mejor y aprender.
hexToRgba
La función hexToRgba, por que la usamos? Parece que lo único que se pretende es añadir el canal alpha al color y eso se puede hacer también en hex sin pasar a rgba. Lo pregunto para aprender.
hexToRgba
Las imagenes de los patros estan codificadas inline in base64 ( 'data:image/svg+xml;base64,PHN2ZyB4bWxuc ). Entiendo que esto es solo para la maqueta, estaría bien quizá crear ahora ya la carpeta donde irán las imagenes.
In my opinion
Disposición en dos columnas
No me acaba de gustar la disposición en dos columnas, 'iguala en visibilidad' a niveles de patrocinio diferente. Lo haría en una sola columna, pero esto es una opinión personal.
itziarZG
left a comment
There was a problem hiding this comment.
Funcionalitat perfect... disseny tambè canviaria el tema de les dues columnes que tot i que entenc que és per a que no quedin massa "buit", no es herarquic...
|
Hola @ctrl-alt-d ! Gracias por tus comentarios.
Ya me dices qué te parece ahora :) Muchas gracias |
ctrl-alt-d
left a comment
There was a problem hiding this comment.
@francescarpi , gracias por tus nuevos commits y tener en cuenta mis comentarios.
He vuelto a revisar, ya se que soy muy pesado lo siento:
-
El contraste de todos los títulos de los niveles (principal, platí, plata, oro, bronze) no es suficiente, las letras deberían ser de color negro. Lo he probado y no quedan mal de color negro. Con las letras en negro el único que no tendría contraste AAA sería el bronze, se puede obviar o se puede poner el color bronze un poco más claro.
-
Lo que hay que arreglar es el contraste del texto 'No hi ha patrocinadors en aquest nivell'
- Esto que escribo ahora ya es un tema personal: yo redondearía las cajas de los niveles,
border-radius: 5pxo parecido.
Por mí, una vez solucionados los problemas de contraste, ya lo veo mergeable a develop. Hay más temas de contraste pero estan en #107, no estan relacionadas con sponsors.
Fixes #80
Nota: El diseño es muy básico y los sponsors cargados son de demo. Este PR añade la funcionalidad básica. En siguientes iteraciones se puede ir mejorando.
Nota 2: Me falta documentar como añadir nuevos sponsors. Pero básicamente es añadir un nuevo markdown.