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

Form

Open Pauline999 opened this issue 2 years ago • 2 comments

Form

Checkbox

Pauline999 avatar Jun 07 '22 13:06 Pauline999

Demo: https://feat-forms-new-spacing-colors--welcome-ui.netlify.app/

theo-mesnil avatar Sep 21 '22 17:09 theo-mesnil

Attention Ă  ne pas aller par la recherche, directement via le menu ;)

theo-mesnil avatar Sep 22 '22 15:09 theo-mesnil

Hola ! petite QA design, n'hésitez pas à me pinguer si des éléments ne sont pas clairs ou si besoin de papoter sur certains points :)

Hint

  • Change color of “default” hint to #[737373](https://convertingcolors.com/hex-color-737373.html)
  • RĂ©duire le line-height Ă  14px

Label

  • RĂ©duire le spacing entre l’icĂ´ne et le label =4px

Field

  • RĂ©duire spacing entre le label et le field = 8px
  • RĂ©duire spacing entre field et hint = 4px

InputText

  • Idem field

Checkboxs

  • RĂ©duire le spacings entre la "Checkbox + label" et le hint = 4px Image
  • RĂ©duire le spacings entre les items dans une liste = 12px Image
  • Augmenter la graisse des labels = Medium

Radio button

  • RĂ©duire le spacing entre le radio button et le label associĂ© = 8px Image
  • RĂ©duire le spacing entre les items radio buttons en liste = 12px + rĂ©duire le spacing entre la liste et le label global de la liste = 12px Image
  • With hint on radio : RĂ©duire le line-height du hint Ă  14px
  • Variants : Il semble y avoir un doublon ici avec la fonction "Disabled" rĂ©pĂ©tĂ©e alors que ça ressemble plus Ă  un variant concernant le status. Les radio button en Ă©tat error ne semblent pas fonctionner au clic.
  • Image

Hook Form

  • Appliquer spacing de 16px entre les components dans un form.

Pauline999 avatar Oct 17 '22 12:10 Pauline999

doublon

Pauline999 avatar Oct 17 '22 12:10 Pauline999

Ok je regarde cela asap merci @Pauline999 :)

theo-mesnil avatar Oct 17 '22 14:10 theo-mesnil

QA sur le com précédent

Hint

  • RĂ©duire le line-height Ă  14px -> pas encore rĂ©alisĂ©, je le remet ici pour pas zapper sur radio/checkbox etc aussi :)

Label

  • RĂ©duire le spacing entre l’icĂ´ne et le label =4px -> KO

Checkboxs

  • RĂ©duire le spacings entre la "Checkbox + label" et le hint = 4px -> KO Image
  • RĂ©duire le spacings entre les items dans une liste = 12px -> KO C'est bien 12px dans le code mais le rendu fais plutĂ´t 24px, si on passe la marge en front Ă  4px(0.25rem) on tombe pile poile sur 12px cotĂ© rendu.
  • Augmenter la graisse des labels = Medium -> KO

Radio button

  • RĂ©duire le spacing entre le radio button et le label associĂ© = 8px -> C'est bien 8px, mais j'ai l'impression qu'il y a un micro margin-right sur le radio de 0.125 rem, si pas necessaire on peut virer pour approcher un peu le label du radio :)
  • RĂ©duire le spacing entre les items radio buttons en liste = 12px + rĂ©duire le spacing entre la liste et le label global de la liste = 12px -> Idem checkbox on peut passer le margin-bottom Ă  4px(0.25rem) pour un rendu 12px en front
  • Variants : Il semble y avoir un doublon ici avec la fonction "Disabled" rĂ©pĂ©tĂ©e alors que ça ressemble plus Ă  un variant concernant le status. Les radio button en Ă©tat error ne semblent pas fonctionner au clic. -> KO doublon de description toujours visible de mon cotĂ© : "Add disabled to the Field component"

Hook Form

  • Appliquer spacing de 16px entre les components dans un form. -> KO Le margin de 16px n'est pas appliquĂ© entre le field email et lacheckbox et la checkbox et le bouton

Pauline999 avatar Nov 10 '22 14:11 Pauline999

Date picker

  • Petite icone agenda sauvage qui se balade Ă  droite du screen :D Image

Clearable (select, field, etc)

  • Utiliser le bouton ghost circle en X-Small (24px x 24px) pour avoir le state rollover moins collĂ© Ă  l'icone -> pour ces sizes : small, medium et marge. J'ai revu le traitement de l'icone sur les maquettes Figma ça portait Ă  confusion ( la taille de l'icone croix reste Ă  16px pas de modification sur le rendu de l'Ă©tat par dĂ©faut, ĂŞtre vigilant par rapport Ă  la marge avec l'arrow du select quand elle est prĂ©sente :)

Image

[WIP sur le reste :) ]

Pauline999 avatar Nov 10 '22 15:11 Pauline999

Testing https://fix-forms--welcome-ui.netlify.app/installation @Pauline999

theo-mesnil avatar Nov 28 '22 16:11 theo-mesnil

Coucou ✌🏼, quelques composants qu'il reste à ajuster sur les précédents feedbacks + Toogle (new) :)

File drop

  • Il manque le dashed sur le contour.

Checkbox

  • Sur la version en prod uniquement il y a un bug sur la couleur de l’icĂ´ne checkbox qui est passĂ©e en blanche
  • Le spacings entre la "Checkbox + label" et le hint est OK par contre j’ai l’impression que ça Ă  mĂ©caniquement rĂ©duit le spacing entre le Checkbox et la Label (ou icone + label) qui est passĂ© Ă  4px alors que c’est 8px.
  • RĂ©duire le spacing entre les items dans une liste = 12px -> KO C'est bien 12px dans le code mais le rendu fais plutĂ´t 24px, si on passe la marge en front Ă  4px(0.25rem) on tombe pile poile sur 12px cotĂ© rendu.

Radio button

  • Margin horizontal alignment entre les radio en ligne = 16px (Column style)
  • Spacing entre le radio button et le Hint = 4px

Clearable (select, field, etc)

  • Impec pour l’update du rollover par contre sur l’input text /select/date picker etc en xs (height Ă  24px) il faut qu’on adapte la taille du hover gris uniquement pour le passer Ă  16px pour Ă©viter qu’il touche le contours du field.

Hook Form

  • Appliquer spacing de 16px entre les components dans un form. > KO Le margin de 16px n'est pas appliquĂ© entre le field email et lacheckbox et la checkbox et le bouton

Toogle

  • Focus state: change the color to #FFE166 et passer la taille du contour de 3px Ă  2px
  • Faire le changement du passage du contour de 3px Ă  2px pour tous les composants field/select/radio etc

Pauline999 avatar Jan 03 '23 08:01 Pauline999