Skip to content

Layout padding prop 지원#954

Merged
DaleSeo merged 3 commits intomainfrom
layout-component-padding-prop
Apr 20, 2026
Merged

Layout padding prop 지원#954
DaleSeo merged 3 commits intomainfrom
layout-component-padding-prop

Conversation

@RiaOh
Copy link
Copy Markdown
Contributor

@RiaOh RiaOh commented Apr 8, 2026

  • Flex, Grid, VStack, HStack 컴포넌트에 padding prop을 추가하였습니다.
  • story docs 양식 통일을 유지하였습니다. (story docs 양식 통일 #952)

테스팅

image - Flex, Grid, VStack, HStack 컴포넌트에 padding 스토리 구동 확인하였습니다.

체크 리스트

  • 코드 리뷰를 요청하기 전에 반드시 CI가 통과하는지 확인해주세요.
  • 컴포넌트나 스토리 변경이 있는 경우 PR에 ui 태그를 달아주세요.
    • UI Tests를 통해 스냅샷 차이가 의도된 것인지 확인해주세요.
    • UI Review를 통해 디자이너에게 리뷰를 요청하고 승인을 받으세요.

@RiaOh RiaOh self-assigned this Apr 8, 2026
@RiaOh RiaOh requested a review from a team as a code owner April 8, 2026 12:44
@RiaOh RiaOh added the ui Chromatic label Apr 8, 2026
@RiaOh RiaOh linked an issue Apr 8, 2026 that may be closed by this pull request
@RiaOh RiaOh changed the title feat: add padding prop in layout components Layout padding prop 지원 Apr 8, 2026
@RiaOh RiaOh marked this pull request as draft April 8, 2026 13:19
@RiaOh RiaOh marked this pull request as ready for review April 8, 2026 16:32
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

추가된 속성에 대한 테스트 추가 부탁드립니다.

Comment thread src/components/Grid/Grid.tsx Outdated
Comment thread src/components/HStack/HStack.tsx Outdated
Comment thread src/components/Flex/Flex.tsx Outdated
Comment thread src/components/VStack/VStack.tsx Outdated
Comment thread src/components/Flex/Flex.stories.tsx Outdated
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

@RiaOh 님, 다음과 같이 차근차근 한 번 리베이스 해보시겠어요? 현재 상태로 병합하면 나중에 커밋 히스토리를 보기가 너무 힘들어질 것 같고, 릴리즈 노트 자동 생성할 때도 문제가 될 것 같습니다.

# 1. 최신 main 브랜치를 먼저 가져옵니다
git fetch origin main

# 2. main 브랜치 기준으로 interactive rebase를 시작합니다
#    에디터가 열리면 남길 커밋만 pick하고, 중복 커밋은 drop합니다
git rebase -i origin/main

# 3. 에디터에서 아래와 같이 정리합니다
#    (중복/불필요한 커밋은 drop으로 변경)
#
#    pick e521802 feat: add padding prop in layout components
#    pick 69092f0 refactor: integration interface comment
#    pick 74d6461 fix: remove padding prop in interface
#
#    ⚠️ 아래 커밋들은 drop합니다:
#    - cad1107 (e521802와 중복)
#    - 84f146b (merge 커밋)
#    - 8754526 (d9e8cf3/69092f0과 중복)
#    - a5e24c6 (다른 분의 codecov 커밋 — main에 이미 있음)
#    - d9e8cf3 (69092f0과 중복)

# 4. 충돌이 발생하면 해결 후 계속 진행합니다
#    git add .
#    git rebase --continue
#
#    rebase를 취소하고 싶다면:
#    git rebase --abort

# 5. rebase가 완료되면 force-push로 원격 브랜치를 교체합니다
#    ⚠️ --force-with-lease는 다른 사람이 push한 변경을 덮어쓰지 않도록 보호해줍니다
git push --force-with-lease origin layout-component-padding-prop

Comment thread codecov.yml Outdated
@RiaOh
Copy link
Copy Markdown
Contributor Author

RiaOh commented Apr 10, 2026

@DaleSeo

@RiaOh 님, 다음과 같이 차근차근 한 번 리베이스 해보시겠어요? 현재 상태로 병합하면 나중에 커밋 히스토리를 보기가 너무 힘들어질 것 같고, 릴리즈 노트 자동 생성할 때도 문제가 될 것 같습니다.

#    ⚠️ 아래 커밋들은 drop합니다:
#    - cad1107 (e521802와 중복)
#    - 84f146b (merge 커밋)
#    - 8754526 (d9e8cf3/69092f0과 중복)
#    - a5e24c6 (다른 분의 codecov 커밋 — main에 이미 있음)
#    - d9e8cf3 (69092f0과 중복)

달레님! 리베이스관련 설명 감사합니다~!
위 중복된 커밋들이 실제로 중복뿐만 아니라 + 추가된 코드도 있고.. 엉망진창이었습니다ㅠ
main브랜치 merge + rebase를 같이 해서 문제가 생긴 것 같아요.
그래서 다시 main브랜치 머지 전으로 돌려서 깔끔하게 rebase했는데, 한번 봐주시면 감사하겠습니다.

@RiaOh RiaOh requested a review from DaleSeo April 10, 2026 05:51
Copy link
Copy Markdown
Member

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

리베이스 성공하셔서 너무 기쁘네요!

@DaleSeo
Copy link
Copy Markdown
Member

DaleSeo commented Apr 20, 2026

@RiaOh 님, PR이 승인되었는데 별 다른 이유없이 너무 오래 열려있는 것 같아서 제가 병합하겠습니다!

@DaleSeo DaleSeo added this pull request to the merge queue Apr 20, 2026
Merged via the queue into main with commit 2728228 Apr 20, 2026
12 checks passed
@DaleSeo DaleSeo deleted the layout-component-padding-prop branch April 20, 2026 01:42
@RiaOh
Copy link
Copy Markdown
Contributor Author

RiaOh commented Apr 21, 2026

@DaleSeo 네, 병합해주셔서 감사합니다!

@github-actions github-actions Bot mentioned this pull request Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui Chromatic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Layout padding prop 지원

4 participants