Skip to content

[Step1] hippo - Component 기본 구조와 JSX - #1

Open
meteorqz6 wants to merge 13 commits into
step1from
hippo-step1
Open

[Step1] hippo - Component 기본 구조와 JSX#1
meteorqz6 wants to merge 13 commits into
step1from
hippo-step1

Conversation

@meteorqz6

@meteorqz6 meteorqz6 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

개인 목표 달성 여부

  • HTML 템플릿을 React 컴포넌트로 변환하는 과정 경험하기
  • 컴포넌트 단위로 UI를 나누는 기준 정하기
  • CSS Module 사용법 익히기

리뷰어에게

특히 봐줬으면 하는 부분, 확신이 없는 코드, 논의하고 싶은 것

  • 7e62f1c: BEM 원칙을 적용하고 CSS Module 표기법의 일관성을 확보하기 위한 커밋입니다.
  • HTML 템플릿을 React 컴포넌트로 변환할 때 어떤 방식으로 진행했는지 공유하면 좋을 것 같습니다.

@meteorqz6
meteorqz6 marked this pull request as draft June 2, 2026 10:15
@meteorqz6 meteorqz6 changed the title Hippo step1 [Component 기본 구조와 JSX] hippo Jun 3, 2026
@meteorqz6
meteorqz6 marked this pull request as ready for review June 3, 2026 00:33
@meteorqz6 meteorqz6 changed the title [Component 기본 구조와 JSX] hippo [Step1] hippo - Component 기본 구조와 JSX Jun 3, 2026

@ehlung ehlung 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.

코드에 직접 남긴 것 외에도 모듈 css와 컴포넌트를 묶어준 부분이 제 코드와 다른 점인 것 같습니다.
src 폴더에 모든 파일이 흩뿌려져 있는 것 같아서 저도 고민이 되었는데, 저 같은 경우에는 음식점 디테일/추가 모달을 컴포넌트는 분리하고 css 파일은 합쳐서 그냥 두었던 것 같습니다.
유성님은 모달의 css 파일을 중복을 감수하고 분리하신 것 같은데, 중복 없이 분리하는 더 좋은 방법이 있을지 이야기해 보면 좋을 것 같습니다!

@@ -0,0 +1,17 @@
import styles from "./Header.module.css";
import addButton from "../../assets/add-button.png";

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.

[배움]
이미지를 import 해오는 방식을 사용하신 부분이 새로워요. 저는 img 태그 안에 경로를 직접 지정해주었는데, 이 방식이 나중에 수정이 되거나 할 때도 더 좋은 것 같아요!

Comment thread README.md
Comment on lines +31 to +33
- `select`의 `required` 속성

폼 제출 시 해당 `select`에서 값을 반드시 선택해야 한다는 유효성 검사 조건을 브라우저에 부여한다. 값이 선택되지 않은 채 제출을 시도하면 브라우저가 자동으로 경고 메시지를 표시하고 제출을 막는다. 이때 `<option value="">선택해 주세요</option>`처럼 `value`가 빈 문자열인 기본 옵션이 있어야 `required`가 정상적으로 동작한다.

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.

[배움]
html을 jsx로 옮기면서 달라지는 부분에만 집중해서 해당하는 부분은 잘 정립이 안되어 있음에도 그냥 넘어갔는데, 덕분에 저도 개념 정리가 되는 것 같습니다!

Comment thread README.md

## 🛠 리팩토링

1. CSS 클래스 네이밍 BEM 원칙 적용

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.

[배움]
BEM 원칙을 적용해서 일관된 규칙으로 통일한 점 좋은 것 같습니다. 저도 리팩토링으로 적용해보겠습니다.

@meteorqz6 meteorqz6 self-assigned this Jun 7, 2026
@coderabbitai

coderabbitai Bot commented Jun 9, 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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 050e8c84-5ac7-406a-bc36-712852a685ea

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hippo-step1

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

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