Skip to content

Eunjin n week01 - 박은진 - #27

Merged
Eunjin-n merged 5 commits into
Eunjin-nfrom
Eunjin-n-week01
Sep 24, 2026
Merged

Eunjin-n merged 5 commits into
Eunjin-nfrom
Eunjin-n-week01

Conversation

@Eunjin-n

Copy link
Copy Markdown
Collaborator

작업 내용

  • week01 내용 구현

변경 사항

  • PokemonHeader 컴포넌트 구현
  • PokemonList 컴포넌트 구현
  • PokemonCard 컴포넌트 구현

스크린샷

  • PokemonHeader 컴포넌트 구현
  • PokemonList 컴포넌트 구현
  • PokemonCard 컴포넌트 구현
image

참고 사항

  • 디자인시스템을 처음 선언해보았는데요. 피그마 네이밍을 그대로 적용해 --color-text-inverse: var(--color-white);로 선언을 했더니, 사용할 때 text-text-inverse처럼 클래스명이 중복되는 형태가 됩니다. 이런 방식으로 사용하는 게 맞는지 의견 부탁드립니다..! 🙏🏻🙏🏻

@Eunjin-n
Eunjin-n requested a review from canofmato September 20, 2026 10:12
@Eunjin-n Eunjin-n self-assigned this Sep 20, 2026
@coderabbitai

coderabbitai Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: f6de0a6e-41fc-4d77-9de9-81fa505cdad9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@canofmato
canofmato changed the base branch from develop to Eunjin-n September 21, 2026 02:37
</button>
</div>
<div className="flex gap-5">
{pokemons.map((pokemon) => (

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

배열을 map으로 렌더링하는 구조가 좋아요. React가 각 항목을 안정적으로 구분할 수 있도록 PokemonCard에 key={pokemon.id}를 추가해 보면 좋을 것 같습니다!

<div className="absolute left-3.5 top-4">
<SearchIcon />
</div>
<input

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

검색 input에 placeholder는 있지만 화면 읽기 사용자가 입력 목적을 안정적으로 확인할 수 있는 label이나 aria-label이 없어요. aria-label을 추가하거나 input과 연결된 label을 사용해 접근 가능한 이름을 부여해 보면 좋을 것 같아요!

@canofmato canofmato left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

은진님 수고하셨습니다! Tailwind 유틸리티와 타입 정의를 함께 활용해 이번 주 학습 목표를 잘 반영하셨네요!👍🏻

@Eunjin-n
Eunjin-n merged commit 0e82cd7 into Eunjin-n Sep 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants