Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

PostRow pin icon is size-guarded only while a post is pinned

オープン
#333 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
1〜2日
初心者へのやさしさ
35/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
next.js, react, typescript
領域
frontend, testing

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

enhancement low-priority

#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.

主要言語
TypeScript
スター
3
フォーク
0
平均マージ
12時間 41分
マージ済み PR(30日)
70

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

decdn/website のほかの issue

decdn/website の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。