bug(tailwind): `pixelBasedPreset` only converts spacing and font size; `rounded-*` and `max-w-*` still emit `rem`
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- tailwindcss, typescript
- Área
- cli, documentation
Línea de trabajo
The preset is defined where pixelBasedPreset overrides theme.extend.fontSize and theme.extend.spacing; read that override first, then compare it with Tailwind's default borderRadius and maxWidth scales. Done means rounded-* and max-w-* also emit pixel values, with the demo at packages/react-email/demo/ab.tsx showing no rem left in the output. Tests should cover the new scales.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What versions are you using? (if relevant)
latest
Describe the Bug
The docs present pixelBasedPreset as the fix for Tailwind's rem units not being supported by many email clients. In [email protected] it only overrides theme.extend.fontSize and theme.extend.spacing, so other utilities that Tailwind expresses in rem still reach the inlined CSS unchanged. The docs do not say the preset is partial, so it is easy to assume everything returned is now in pixels.
What is affected (leave empty if unsure)
Tailwind Component
Link to the code that reproduces this issue
https://github.com/SwissArmyBud/react-email/tree/aporter/2611-rem-from-pixelbasedpreset-demo
To Reproduce
A runnable before/after demo lives on a separate branch so it stays out of this PR:
aporter/2611-rem-from-pixelbasedpreset-demo.
Its README walks through it: cd packages/react-email && npx tsx demo/ab.tsx renders the same email with and without the new scales and prints the rem values found in each, all to local HTML for review. The branch also holds captured evidence (evidence-02.txt to evidence-07.txt: the published-package baseline, per-class output, the scan of Tailwind's rem scales, and the mutation check for the tests) and pre-generated a/b output in the demo folder.
Expected Behavior
The docs present pixelBasedPreset as the fix for Tailwind's rem units not being supported by many email clients, so it is easy to assume everything returned is now in pixels.
What's your node version? (if relevant)
24/LTS
- Lenguaje dominante
- TypeScript
- Estrellas
- 19.9k
- Forks
- 1.1k
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 61
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de resend/react-email
-
Type: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
resend/react-email#3889 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
resend/react-email#3878 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Tailwind stacked media variants leave nested CSS in rendered emailsPosiblemente ocupada @lakshya-dhariwal la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
resend/react-email#3874 ·
Los mantenedores suelen responder en 1 día
-
Inspector Text panel omits Uppercase mark toggle (MARK_NAMES missing "uppercase")Posiblemente ocupada @svargas-dev la tomó hace 9 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 22/100
resend/react-email#3849 ·
Los mantenedores suelen responder en 1 día
-
Dependency DashboardAbiertodependencies
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
resend/react-email#746 ·
Los mantenedores suelen responder en 1 día
Todos los issues de resend/react-email
Issues similares
-
Language request: NixAbiertoenhancement language
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
pngwn/twinkleplop#178 ·
-
build P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
maniator/verticopolis#1945 ·
Los mantenedores suelen responder en 1 día
-
Admin users page number stays stale after filteringPosiblemente ocupada @matthewxkong la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
feature request needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
outline/outline#14529 · 1 comentario ·
Los mantenedores suelen responder en 1 día