search

cozy / cozy-ui

React components and CSS styles for Cozy apps
https://cozy.github.io/cozy-ui/react/
MIT License
48 stars 37 forks source link

Fix colors for intents and height layout for mobile #2661

Closed JF-Cozy closed 4 months ago

JF-Cozy commented 4 months ago

J'ai testé le correctif sur le layout en mobile avec Drive, MesPapiers, Contacts et Store.

Le correctif sur Modal est dû au fait qu'on s'en sert encore (à tort car deprecated depuis longtemps) pour les Intent

Pour la gestion du type/variant du thème, si on manipule le body ou une balise très haute comme ça, on devrait après réflexion avoir 2 composants, ou alors un prop pour distinguer l'usage du type et l'usage du variant. Car le type ne se définit qu'une fois pour toute (pour l'app et pour l'intent), alors que le variant peut être utilisé à l'intérieur de l'app. Si c'est un seul composant qui fait les deux on devrait avoir une prop <CozyTheme master={true}> par exemple qu'on utiliserait uniquement dans les providers de l'app/intent et qui s'occuperait de gérer le type du thème.

demo: https://jf-cozy.github.io/cozy-ui/react/#/IntentIframe

zatteo commented 4 months ago

Is it expected that in dark mode the text in the intent is in black on the demo ?

JF-Cozy commented 4 months ago

Is it expected that in dark mode the text in the intent is in black on the demo ?

tiens je m'attendais pas à ce que ça fonctionne pour le fond, ce n'est pas le cas en dev 🤔 je vais regarder

cozy-bot commented 4 months ago

:tada: This PR is included in version 110.1.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: