PostRow pin icon is size-guarded only while a post is pinned
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- next.js, react, typescript
Línea de trabajo
Read components/blog/PostRow.tsx, test-utils/react-tree.ts, and scripts/check-out.mjs check 8 to understand the pin rendering and existing SVG coverage. The issue describes adding a unit test when PostRow gets tests for other reasons, or moving the pin into components/ui/icons.tsx for coverage by icons.test.ts; completion means the pin’s size attributes are covered even when no post is pinned.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
#330 added width/height attributes to every inline <svg> so that a paint before the stylesheet applies can't draw them at full container width. scripts/check-out.mjs check 8 enforces that against every prerendered <svg> in out/.
The pin glyph in components/blog/PostRow.tsx renders only for a pinned post. Today that is content/blog/09-testnet-is-live.mdx, and its pin is the only <svg> on out/blog/index.html. If no post is pinned, check 8 sees nothing on the blog. A regression in the pin's attributes would then go unnoticed until a post is pinned again.
A unit test needs a vi.mock("next/link"), because findAll in test-utils/react-tree.ts expands function components and next/link's LinkComponent calls hooks. That's moderate cost for a low-risk glyph, so it's worth doing when PostRow gets a test for other reasons, or if the pin moves into components/ui/icons.tsx, where icons.test.ts would cover it automatically.
- Lenguaje dominante
- TypeScript
- Estrellas
- 3
- Forks
- 0
- Merge medio
- 12 h 41 min
- PR fusionados (30 d)
- 70
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 decdn/website
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Los mantenedores suelen responder en 1 día
-
bug med-priority
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 4/5 1-2 días Aptitud para principiantes 54/100
Los mantenedores suelen responder en 1 día
-
enhancement low-priority
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
decdn/website#268 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de decdn/website
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
db-ux-design-system/core-web#8613 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
foxglove/mcap#1907 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug internal-plugin pr-welcome upstream
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
dsh-tauri-desk/deepseek-harness-desktop#1225 · 1 comentario ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
unjs/unhead#1027 · 1 reacción ·
Los mantenedores suelen responder en 1 día