[DR] Buttons - Githubissues

Cl0v1s commented 11 months ago

Les boutons sont prêts pour une revue. Tu peux aller les voir sur ta machine et jouer avec. Pour ce qui concernent ceux avec des Icones dedans, c'est déjà ok, faut simplement que je code les icones, mais ça fait partie du contenu du bouton, pas de son apparence en général.

http://localhost:9005/Actions/Button/index.html

Je t'invite donc à me lister tes retours ci-dessous, ou carrément me filer du css !

Dehelssey commented 10 months ago

les boutons en size-50 doivent avoir un radius de 8px pas de 16

les boutons primary default pourquoi ils sont vert clair ? normalement la couleur change pas au survol, juste le box-shadow

aussi faut foutre les transitions sur l'élément natif, pas sur son état hover, comme ça y'a transition au survol et en sortie de survol.

À part ça c'est good to me!

Dehelssey commented 10 months ago

désolé pour le délai, je suis sous l'eau au taf et le lien marchait plus, j'ai du rebuild^^

Cl0v1s commented 10 months ago

Pas de soucis, si ça parait bizarre, c'est que je suis passé sur tes appels aux boutons et essayé de comprendre ce que je pouvais haha

Je reprends ça.

D'ailleurs on peut voir que à ce stade, les boutons ont pas d'état disabled

Cl0v1s commented 10 months ago

DR ok !

Et visible sur le lien du README :)

Cl0v1s commented 10 months ago

Close quand c'est ok pour toi

Dehelssey commented 9 months ago

concernant l'état disable pour les boutons -> https://www.smashingmagazine.com/2021/08/frustrating-design-patterns-disabled-buttons/

si on décide qu'on en a quand même besoin je peux te faire une propale yes

concernant les boutons taille 50 le drop shadow est OK
pour les boutons taille 100 en revanche, comme pour les input, t'es resté sur les mêmes valeurs de drop shadow alors qu'elles sont grossies (pour conserver une sorte de proportionnalité) il faudrait var(--dp-75) par défaut et var(--dp-100) au hover (au lieu de 25 / 75 actuellement)
Concernant le bouton secondaire, son hover doit être le même que le bouton primaire (vert et drop shadow 100)

Cl0v1s commented 9 months ago

Je comprends le point de cet article, à nous de bien conçevoir les écrans de manière à ne pas rencontrer ces soucis.

Néanmoins, je pense qu'on en aura besoin à un moment où à un autre, on fait un design-system :) Idem pour tous les composants en fait

Cl0v1s commented 9 months ago

C'est traité !

Dehelssey commented 9 months ago

le bouton secondary 50 n'a pas de bordure au hover la version 100 n'a ni border ni radius ^^

Cl0v1s commented 9 months ago

Alors j'ai pas ce soucis :0

Tu as le nom de ton navigateur / sa version ?

Dehelssey commented 8 months ago

Pulse Brower v. 1.0.0-a.87 👀

Et je confirme que j'ai ce problème sur que les versions "secondary" du bouton

Cl0v1s / Canaille

[DR] Buttons #1