Numeric 'scale' and 'aspectRatio' styles get a 'px' suffix (React treats them as unitless)
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 88/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- react, typescript
- Domaine
- frontend
Piste de recherche
L’issue renvoie à UNITLESS_STYLE dans packages/redact/src/dom/dom.ts. Lis cette liste et vérifie comment les valeurs numériques des styles inline sont sérialisées. Cherche ensuite les tests couvrant les styles sans unité. Ajoute les deux propriétés nommées et vérifie que les valeurs numériques de scale et aspectRatio sont rendues sans suffixe px.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
Redact adds px to a numeric scale or aspectRatio inline style. React does not. The result is invalid CSS, so the browser ignores the declaration.
<span style={{ scale: 0.25, aspectRatio: 1.5 }} />
style.scale |
style.aspectRatio |
|
|---|---|---|
| React 19 | 0.25 |
1.5 |
| Redact 0.2.0 | 0.25px (ignored) |
1.5px (ignored) |
There is no error or warning. The element just renders without the scale or the aspect ratio. In our app, this silently broke an icon cross-fade animation (scale: visible ? 1 : 0.25) and a size preview box (aspectRatio: width / height).
Cause
UNITLESS_STYLE in packages/redact/src/dom/dom.ts does not contain scale or aspectRatio. React's unitless list (react-dom-bindings/src/shared/isUnitlessNumber.js) contains both.
Suggested fix
Add aspectRatio and scale to UNITLESS_STYLE.
React's list also contains vendor-prefixed names that UNITLESS_STYLE does not match, for example WebkitLineClamp, WebkitFlex, WebkitBoxFlex, MozBoxFlex, msFlex and msGridRow. A numeric WebkitLineClamp: 2 (a common pattern for multi-line truncation) probably has the same problem. I did not test the prefixed names.
Versions
@tanstack/redact0.2.0.UNITLESS_STYLEonmain(d56d87cb) has the same gap.- Compared against
react-dom19.3.0.
- Langage dominant
- TypeScript
- Étoiles
- 408
- Forks
- 13
- Merge moyen
- 1 h 44 min
- PR mergées (30 j)
- 21
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de TanStack/redact
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
TanStack/redact#67 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Bug: jsx runtime doesn't apply defaultProps (breaks Recharts 2: "Could not find xAxis by id")Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
TanStack/redact#58 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de TanStack/redact
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Graphify-Labs/graphify#4860 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
sveltejs/eslint-plugin-svelte#1615 ·
Les mainteneurs répondent en général sous 7 jours
-
Parallel reticle_verify opens per-flow leases without the source lease's ignoreHTTPSErrorsPeut-être pris @as992949791 l’a pris aujourd’hui. Ouvertegood first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
reticlehq/reticle#2033 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour