gcba / estandares

Estándares y lineamientos de experiencia digital
https://gcba.github.io/estandares/
MIT License
16 stars 5 forks source link

Enlaces #241

Closed paulajoglar closed 1 year ago

paulajoglar commented 1 year ago

Enlace

Nombre en Obelisco Enlace

Tipo de componente Átomo

Links UI Kit | Branch | Ficha Figma | Estándares | Storybook | Informe | Excels

Ejemplo en Bootstrap Link

Ejemplos de aplicación en BA

marusaad commented 1 year ago

Vemos que el componente está actualmente en una branch y no en el UI Kit. Se va a actualizar para la utilización en Lista informativa.

marusaad commented 1 year ago

Se trabajó sobre la branch de "Enlace"

Algunas observaciones @camilembo

Enlace

camilembo commented 1 year ago

Maruuu! Yo tampoco entiendo muy bien el tema del estado activo, porque entiendo que un enlace al hacerle click solo te redirige hacia otra pantalla o te realiza una descarga por ejemplo, entonces el estado activo entiendo que no se vería nunca. Pero me puse a comparar con los botones, que siento que tienen una función bastante similar al enlace y también tenemos un estado activo. Me pregunto entonces si ese "activo" no funcionaría como el "pressed" en Material design, intentando encontrarle una justificación eeh, pero no estoy segura. Vos que opinás? si querés lo vemos en una reu rápida :D

marusaad commented 1 year ago

@camilembo Estuve chequeando storybook y la branch. Al parecer hay algunas diferencias para resolver

Estados en el UI Kit

Estados en storybook

Acá nuestro "activo" podría referirse al visitado, pero no tendría sentido que tenga un stroke como el "en foco" Además, tenemos otros colores que los que están en storybook para los enlaces

Pasando en limpio

camilembo commented 1 year ago

Usamos para revisar y hacer la ficha de enlaces la brunch que ya está creada en Figma. La misma no coincide con lo que hay actualmente en estándares así que hay que revisar tanto texto como variantes que ya revisamos junto con desarrollo. https://www.figma.com/file/Pi8vXNQiMDtCNvV1aTCQX7/branch/oW1OTV1jupHhVRlaOVt3Ff/Fichas?type=design&node-id=1400%3A1255&mode=design&t=ZtX5E72MoCEeC5Dv-1

marusaad commented 1 year ago

Se cargó la investigación realizada al archivo de Análisis de componentes en la page Enlaces. Incluye research de otros design systems y lo que se habló con desarrollo respecto a un futuro cambio de estilo de focus en los enlaces de texto.

Como cambio para realizar por ahora, el estado focus de los enlaces de texto:

Enlace (2)

marusaad commented 1 year ago

👩🏻‍🔧 Se terminó la construcción del componente y sus variantes. Queda para chequear @camilembo. 👩🏻‍🔧 Se armó un mini handoff con todos los estados y tamaños correspondientes para que sea más fácil para el equipo de desarrollo. 👩🏻‍🔧 Se prototipó el componente: el estado sobre con "while hovering" y el estado focus con "key/gamepad" con la tecla Tab. Creo que no se ve bien cuando pongo play en el prototipo, así que o se cambia a "while pressing" o no se deja prototipado el focus.

Notas

Enlace (3)

Handoff - Enlace

marusaad commented 1 year ago

@camilembo listo para revisión.

marusaad commented 1 year ago

Lista la ficha de enlaces @camilembo para revisión.

Enlace (5)

camilembo commented 1 year ago

Genial @marusaad la veo bien la ficha! Lo pasamos a listo para maquetar 🥇

marusaad commented 1 year ago

Se creó una variante del componente para fondos oscuros (se usa en banner y en footer). El color del enlace pasa a ser "activo" (blanco).

Image

marusaad commented 1 year ago

Se agregó la variante para fondo oscuro en la documentación de la ficha

Image

camilembo commented 1 year ago

Se hizo merge el día 21/09/2023

Diehault commented 1 year ago

Abrimos el ticket para actualizar el componente y ajustar propiedades.

¿Qué se hizo?

Sumamos una instancia/propiedad de texto al componente que nos permite editar el texto sin la necesidad de entrar específicamente en la capa.

¿Cómo quedaron las propiedades?

Image

Con esto también se actualizaría automáticamente el componente de "lista de enlaces", teniendo la posibilidad de editar el wording de cada enlace con mayor facilidad.

camilembo commented 1 year ago

Gracias Die! Se hizo merge con este ajuste el 29/09 :)

Diehault commented 4 months ago

¡Hola! 📎

Actualizamos el componente aprovechando la actualización general de la guía tipográfica.

¿Qué hicimos?

¿Qué se tiene que hacer tras el cambio?

¿Cómo se ve la página ahora en la librería del UI kit?

Image

marusaad commented 2 months ago

📢 Nuevo requerimiento

Tickets relacionados

311

Qué

Actualizar el componente según el nuevo estilo de grillas de la v.2 de Obelisco.

Por qué

Para estandarizar tamaños, modulación y visualizaciones de todos los componentes.

Quién lo va a trabajar

Diseñadores

Quién lo va a revisar

PM + SGOs

Criterios de aceptación

🕰️ Changelog

No se realizaron cambios en esta revisión.

✏️ Notas

No hay ninguna nota para mostrar.

marusaad commented 3 weeks ago

📢 Nuevo requerimiento

Issues relacionados

300

Qué

Actualizar los estilos de color del componente aplicando las variables correspondientes a la v2 de Obelisco.

Por qué

Porque posiciona al sistema en un nivel de calidad superior al cumplir con los criterios de accesibilidad de la WCAG, y facilita la adopción, la personalización y la escalabilidad a nivel global.

Quién lo va a trabajar

Diseñadores

Quién lo va a revisar

PM + SGOs

Criterios de aceptación

🕰️ Changelog

CHANGELOG Descripción
Agregado Creamos el estado visited-ondark con un purple más claro, generando consistencia con el estado visited en fondos claros.
Arreglado
Eliminado
Obsoleto
Cambiado Ya no hay cambio de color entre los estados default y hover. Plantear un cambio de color es muy poco posible dentro de los lineamientos de WCAG que solicitan un ratio no sólo entre el color de texto y el fondo, y el color de texto del enlace con el color del texto del párrafo, sino que también se pide un ratio a cumplir entre estados. Que, en nuestro caso, con un azul un poco más oscuro, no era posible cumplir. Por lo tanto para pasar el criterio de WCAG no nos basamos en el color para indicar el estado de hover, sino en un cambio en el grosor del subrayado, que en default está en 2px y en hover en 3px.

✏️ Notas

Por las posibilidades constructivas de Figma, no podemos personalizar el subrayado como sí lo podemos hacer en el código. Es por esto, que en Figma se mostrarán estos enlaces a modo representativo y sólo en su modalidad standalone (o sea, un enlace que no está dentro de un párrafo, como puede ser dentro de un breadcrumb o de una lista de enlaces o destacado). En el caso de tener que utilizar un enlace en un párrafo, se tendrá que recurrir a subrayar el enlace con la propiedad de texto (O sea, Cuerpo M Subrayado y Cuerpo S Subrayado) y aplicarle el color que corresponda. Vamos a tener que enfocarnos en documentar y explicar bien todo esto para no confundir a los diseñadores. Figma anunció en sus Release Notes que pronto se efectuará la personalización de subrayado.

marusaad commented 2 weeks ago

En base al release de Figma y la posibilidad de customizar los estilos de subrayado, actualizamos el componente de enlace standalone. Para el enlace inline creamos estilos de texto de "Enlace" para cuerpos y titulares.