Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Text area : text-state Value - Height adaptation broken #2154

Open
Pauline999 opened this issue Jul 31, 2023 · 6 comments
Open

Text area : text-state Value - Height adaptation broken #2154

Pauline999 opened this issue Jul 31, 2023 · 6 comments
Assignees

Comments

@Pauline999
Copy link

Pauline999 commented Jul 31, 2023

For which Figma library?

Forms

For which component?

Textarea

Current Behavior

  1. Le composant est cassé (WYSUWYG aussi) quand on change le text-state : de Placeholder à Value le texte reste sur une seule ligne. Le component _Input / Content perd l'attribut "fill" sur la hauteur.
  2. Aussi lorsqu'on ajoute une icône ou image celle ci se centre sur la hauteur

Capture d’écran 2023-07-31 à 16 45 03
Capture d’écran 2023-07-31 à 16 47 35
Capture d’écran 2023-07-31 à 16 47 39

Expected Behavior

  1. Le texte soit s'adapter sur la hauteur et revenir à la ligne automatiquement. _Input / Content doit garder l'attribut "fill" sur la hauteur.
  2. Garder l'icone ou l'image positionnés en haut à gauche

Capture d’écran 2023-07-31 à 16 52 25

Screenshots / Videos

No response

@Pauline999
Copy link
Author

Pauline999 commented Aug 29, 2023

To Review :

  • The text adapts to the height and wraps automatically
  • Keep the icon or image positioned at the top left
  • ⚠️ Updates can also impact "Fields" and "Selects" components because of _Input / Content component that is in common

Can you check component please before 30/08 night ?
Figma Text Area component is here

Thanks ✌🏼
Mandatory
@VictorWTTJ

Optional
@Jademhm
@aurelienalletru
@Louiseaks

@aurelienalletru
Copy link
Contributor

@Pauline999 lorsque je suis sur la props Value et que je change avec/sans icon alors le texte dans n'est pas le même.

Screenshare.-.2023-08-30.2_04_54.PM.mp4

@aurelienalletru
Copy link
Contributor

Le reste semble okay ✅ :) merci @Pauline999

@Pauline999
Copy link
Author

Pauline999 commented Aug 30, 2023

Merci, bien vu ! c'est fixey @aurelienalletru ;). J'ajoute un petit ticket dans le backlog sur ce sujet pour repasser sur les tous les composants des forms pour les améliorer afin qu'au changement de props on ai plus à retaper le texte 🙌🏼

@VictorWTTJ
Copy link

Petit problème lors du resize. Rien de drama, mais j'ai pas réussi à identifier le pourquoi du comment…

Enregistrement.de.l.ecran.2023-09-04.a.10.04.49.mov

@Pauline999
Copy link
Author

Merci, j'ai pu corriger le problème ✅ , ça venait d'une différence de 1px sur la largeur d'un des variants..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Status: To mitigate
Development

No branches or pull requests

3 participants