search

Cl0v1s / Canaille

Design system for BDX.town projects
0 stars 0 forks source link

[DR] TextInput #3

Open Cl0v1s opened 10 months ago

Cl0v1s commented 10 months ago

Le TextInput est prêt https://cl0v1s.github.io/Canaille/?story=text-input--text-input Pour ce qui est de la taille 50 elle s'adapte en fait à la taille du parent (pour reproduire le comportement du petit textinput dans ta maquette) @Dehelssey

Dehelssey commented 9 months ago

de ce que je vois c'est la taille 100 qui s'adapte 🤔 taille 50 y'a plus de label c'est normal ?

Sinon les drop shadows t'as pas les mêmes tailles que sur ma maquette, faudrait var(--dp-75)au hover et var(--dp-100) au focus (au lieu de 25 / 75 actuellement)

D'ailleurs dans tes var(--dp-XX) tu utilises la couleur en hexa en dur, ce serait mieux d'utiliser la var de la teinte non ? genre var(--border-color) qui existe déjà :)

Cl0v1s commented 9 months ago

J'avais mal lu les maquettes je ferai un smallInput à part, j'ai viré la taille 50 du coup !

Je suis pas sûr de comprendre ta question :) Chaque composantes des couleurs est automatiquement disponible avec les suffix -h -s -l grâce à ce bout de code !

https://github.com/Cl0v1s/Canaille/blob/main/src/scss/_variables.scss#L332

Dehelssey commented 9 months ago

je sais pas, j'avais l'impression d'avoir lu des valeurs couleurs en hexa mais effectivement là tout est en variables 🤔

Du coup RAS

enfin si, faudrait ptet override la couleur du cadre de focus géré par le navigateur non ? ça fait un cadre dans le cadre là, avec la manière dont sont gérés tes composants (fiedlset / label / input)

je veux pas niquer l'accessibilité mais c'est chiant que l'outline se place sur l'input et pas le fieldset^^

Cl0v1s commented 9 months ago

Tu as des guidelines générales concernant les outlines ?

-- Sent from Canary (https://canarymail.io)

On vendredi, janv. 05, 2024 at 11:44 AM, Seth Pilgrim @. @.)> wrote:

je sais pas, j'avais l'impression d'avoir lu des valeurs couleurs en hexa mais effectivement là tout est en variables 🤔

Du coup RAS

enfin si, faudrait ptet override la couleur du cadre de focus géré par le navigateur non ? ça fait un cadre dans le cadre là, avec la manière dont sont gérés tes composants (fiedlset / label / input)

je veux pas niquer l'accessibilité mais c'est chiant que l'outline se place sur l'input et pas le fieldset^^

— Reply to this email directly, view it on GitHub (https://github.com/Cl0v1s/Canaille/issues/3#issuecomment-1878467878), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AAHAADXQPLUNZS5NOR77XVTYM7KPTAVCNFSM6AAAAABBBRLDC2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGQ3DOOBXHA). You are receiving this because you authored the thread.Message ID: @.***>

Dehelssey commented 9 months ago

pas spécialement. je sais juste qu'il faut qu'il y ait un indicateur de focus quelqu'il soit. Techniquement je l'ai prévu avec l'augmentation de la drop-shadow MAIS j'ai un doute sur le fait que ce soit suffisant niveau accessibilité...

Je couplerai bien ça à un changement de background du champ OU de couleur de bordure ?

Je vais cogiter