PostRow pin icon is size-guarded only while a post is pinned
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- next.js, react, typescript
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
#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.
- Lingua principale
- TypeScript
- Stelle
- 3
- Fork
- 0
- Merge medio
- 12h 41m
- PR unite (30g)
- 70
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di decdn/website
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
I maintainer di solito rispondono entro 1 giorno
-
bug med-priority
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 4/5 1-2 giorni Idoneità per principianti 54/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement low-priority
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
decdn/website#268 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di decdn/website
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
zts212653/clowder-ai#1678 ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
Progressive-Victory/the-website#680 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100