angular / angular/components

perf(cdkTextareaAutosize): Skip height measurements when textarea value is empty

Ouverte
#24,210 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
area: cdk/text-field G P3
Langage dominant
TypeScript
Étoiles
25k
Forks
6.8k
Merge moyen
1 j 8 h
PR mergées (30 j)
91

Description

### Is this a regression?

- [ ] Yes, this behavior used to work in the previous version

### The previous version in which this bug was not present was

_No response_

### Description

Calls to _measureScrollHeight, which trigger css layout, are not needed when there's nothing in the text field. In this case, simply set the height to 1 row (height of one line).

See internal CL 421826236 for more context.

### Reproduction

Steps to reproduce:
1. Bootstrap a textarea with cdkTextareaAutosize that has value of '' (empty)

### Expected Behavior

_measureScrollHeight() called only when there's a non-empty value.

### Actual Behavior

_measureScrollHeight() is called regardless of value.

### Environment

- Angular: 13
- CDK/Material: 13
- Browser(s): All
- Operating System (e.g. Windows, macOS, Ubuntu): All

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par l’implémentation de cdkTextareaAutosize et suivez le point d’entrée _measureScrollHeight en utilisant un textarea dont la valeur est vide. Vérifiez que les valeurs vides utilisent une hauteur d’une ligne sans mesure, tandis que les valeurs non vides conservent le comportement de mesure ; ajoutez ou mettez à jour le test pertinent si l’emplacement du test de l’implémentation est identifié.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
angular, typescript
Domaine
frontend, performance
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.