cozy-ui icon indicating copy to clipboard operation
cozy-ui copied to clipboard

Missing portage color into palette.styl

Open cballevre opened this issue 2 years ago • 4 comments

Into cozy-sharing, we had a problem on sharing badge with me because portage color has been remove from cozy-ui (cc: https://github.com/cozy/cozy-ui/commit/5fd473af94b9febeef8523bdc42375930cdea11c). As a temporary fix, I hardcoded the color into styl file (cc: https://github.com/cozy/cozy-libs/pull/1957).

Now the question is how to fix it properly ?

Adding it back to palette.styl because the color is still present into palette.json or @JF-Cozy do you have any other ideas ?

cballevre avatar Jan 09 '23 10:01 cballevre

Few related issues: https://github.com/cozy/cozy-ui/issues/1860 https://github.com/cozy/cozy-ui/issues/1546

Crash-- avatar Jan 09 '23 14:01 Crash--

Voir la réponse : https://github.com/cozy/cozy-libs/pull/1957#pullrequestreview-1240656229

Est-ce que ce ne sont pas des cas sans spec ou basés sur des composants legacy ?

JF-Cozy avatar Jan 09 '23 15:01 JF-Cozy

Si et on va supprimer cette dite valeur dans pas longtemps. Mais, reste que portage est défini dans la palette.json https://github.com/cozy/cozy-ui/blob/2746d0f5e7223f736586f73f7dca1d7f7ec54af3/stylus/settings/palette.json#L36 mais qu'elle n'est pas présente dans la palette.styl https://github.com/cozy/cozy-ui/blob/2746d0f5e7223f736586f73f7dca1d7f7ec54af3/stylus/settings/palette.styl

Et j'ai perdu le fil de ce qu'on faisait de ces 2 palettes et qu'elle était la vision sur ce sujet.

Crash-- avatar Jan 10 '23 08:01 Crash--

@cballevre I found the way to fix that correctly: https://github.com/cozy/cozy-libs/pull/2041

Crash-- avatar Mar 01 '23 08:03 Crash--