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

Fix inverse state in tags #877

Closed
5 tasks done
yceballost opened this issue Oct 25, 2022 · 4 comments · Fixed by Telefonica/mistica-web#661 or #1059
Closed
5 tasks done

Fix inverse state in tags #877

yceballost opened this issue Oct 25, 2022 · 4 comments · Fixed by Telefonica/mistica-web#661 or #1059

Comments

@yceballost
Copy link
Collaborator

yceballost commented Oct 25, 2022

Las tags sobre imágenes en dark mode se rompen.

Aprovecharía para cambiar quizás la definición actual de los estados inverse en light. @AnaMontes11 comentó en algún momento que aplicaría el mismo estado que el no-inverse en el inverse y no cambiar a blanco puro en estado inverse.

@yceballost yceballost added this to the 6.0.2 milestone Oct 25, 2022
@yceballost yceballost changed the title Tag inverse state Fix tag in inverse state Oct 25, 2022
@yceballost yceballost changed the title Fix tag in inverse state Fix tag over images with inverse state Oct 25, 2022
@aweell aweell self-assigned this Oct 28, 2022
@AnaMontes11
Copy link
Contributor

AnaMontes11 commented Nov 4, 2022

Tras varias pruebas concluimos realizar los siguientes ajustes:

Light Mode:

  • Ajustar las inverse para que se muestren igual que las no-inverse

Las no-inverse las mantenemos sin tocar.

Dark Mode:

  • Las inverse se mostrarán con background blanco sólido igual que las inverse actuales en light.

Las no-inverse las mantenemos sin tocar. Una posible mejora a futuro (a falta de hacer las pruebas necesarias), es que el background en vez de llevar en successLow en white con 5% de opacidad lleve successHigh con 5% de opacidad. Y así con todas. De manera que quedan más integradas.

@AnaMontes11
Copy link
Contributor

AnaMontes11 commented Nov 4, 2022

Update mobile library*
Update desktop library*
Update UI Kit & Materials*

*Las tags en las librerías en Figma no las tocamos por el momento. A futuro podemos valorar añadir unas props específicas para dark mode.

@AnaMontes11 AnaMontes11 changed the title Fix tag over images with inverse state Fix inverse state in tags Nov 4, 2022
@yceballost yceballost modified the milestones: 6.2.0, 6.3.0 Nov 14, 2022
@aweell
Copy link
Collaborator

aweell commented Nov 23, 2022

Completed waiting for web to update documentation

@AnaMontes11
Copy link
Contributor

Reabrimos esta issue debido a que en nativo no se pueden aplicar constantes light en dark.
Como solución se plantea crear nuevas constantes "inverse" para light y dark.

@yceballost yceballost moved this from In progress to Done in Mística Design System Feb 9, 2023
@yceballost yceballost modified the milestones: 7.3.0, 7.2.0 Feb 9, 2023
@yceballost yceballost moved this from Done to Released in Mística Design System Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment