Skip to content

fix: Stack 컴포넌트 props_to_style 형태로 변경#53

Merged
Obj277 merged 6 commits intomainfrom
fix/stack-css
Jul 1, 2025
Merged

fix: Stack 컴포넌트 props_to_style 형태로 변경#53
Obj277 merged 6 commits intomainfrom
fix/stack-css

Conversation

@Obj277
Copy link
Contributor

@Obj277 Obj277 commented Jul 1, 2025

작업내역

고민사항 / 중점적으로 리뷰받고 싶은 부분

꼭 확인해주세요

  • PR 작성자
    • 자체테스트 혹은 QA 가 완료되었나요?
    • 서버 배포 필요여부 및 배포 순서를 확인하였나요?
    • Squash merge를 해주세요
    • 리뷰 대응중인 경우 PR의 상태를 꼭 draft로 변경해주세요
    • 코드 설명이 필요하다 싶은 부분은 comment를 통해 설명을 붙여주세요
    • 테스트 코드를 테스트 하기 위해서는 test라벨을 붙여서 PR을 생성 해주세요
  • 리뷰어
    • 꼭 반영해주었으면 하는 부분은 request change로 남겨주세요
    • 리뷰 전에 확인이 필요하거나 궁금한 부분은 comment로 남겨주세요
    • 반영되면 좋지만 꼭 반영되지 않아도 되는 코멘트인 경우에는 approve로 남겨주세요

@Obj277 Obj277 self-assigned this Jul 1, 2025

return (
<div ref={ref} className={combined_classes} style={spacing_style} {...props}>
<div ref={ref} className={className} style={combined_style} {...props}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

요거 이슈원인 해결된거같아서 forwardRef 넣어두 될거같은데 검토한번만 부탁드려도될까요? 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 그래서 forwardRef 다시 넣어뒀어요 🌝 @sunginHwang

@Obj277 Obj277 requested a review from sunginHwang July 1, 2025 02:59
@Obj277 Obj277 merged commit f07797f into main Jul 1, 2025
2 checks 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.

3 participants