Skip to content

fix: HOTFIX DESIGN 수정#390

Merged
heesunee merged 10 commits intodevelopfrom
fix/#388/hotfix-design
Sep 8, 2025
Merged

fix: HOTFIX DESIGN 수정#390
heesunee merged 10 commits intodevelopfrom
fix/#388/hotfix-design

Conversation

@heesunee
Copy link
Contributor

@heesunee heesunee commented Sep 8, 2025

#️⃣ Related Issue

Closes #388

  • 회원가입, 홈

Summary by CodeRabbit

  • 스타일
    • 회원가입 페이지 레이아웃 개선: 컨테이너의 고정 높이를 제거하고 내부 섹션이 가변적으로 확장되도록 조정해 다양한 화면 크기에서 더 안정적이고 자연스러운 배치를 제공합니다. 스크롤 및 뷰포트 대응이 향상되었습니다.
    • 배너 카드 버튼 최소 높이 도입: 버튼 크기 일관성과 터치 영역을 확보해 가독성과 접근성을 개선했습니다. 시각적 균형이 좋아지고 카드 콘텐츠와의 정렬이 더 안정적입니다.
    • 기능 동작 변화는 없습니다.

heesunee and others added 10 commits September 5, 2025 13:11
Co-authored-by: CHAEEUN KIM <154000318+bongtta@users.noreply.github.com>
* fix: (QA/3) 홈으로 페이지 이동할 때 배너 이미지 로드 전에 텍스트만 뜨는 문제 해결  (#384)

* fix: 홈으로 페이지 이동할 때 배너 이미지 로드 전에 텍스트만 뜨는 문제 해결 (#384)

* fix: 배너 최소 높이 설정 (#384)

* fix: 회원가입 design, gaEvent 수정 및 매칭 현황 내 중복 데이터 오류 해결 (#383)

* fix: sign-up 페이지 배경색 짤리는 오류 수정 (#383)

* fix: match-card-view gaEvent 삭제 (#383)

* fix: gaEvent 위치 수정 (#383)

* fix: 캐시가 섞이면서 데이터 중복되는 에러 해결 (#383)

* fix: 주석 제거 (#383)

---------

Co-authored-by: heesunee <heesun729@uos.ac.kr>

---------

Co-authored-by: Yewon Kim <163109964+yeeeww@users.noreply.github.com>
Co-authored-by: heesunee <heesun729@uos.ac.kr>
@coderabbitai
Copy link

coderabbitai bot commented Sep 8, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

회원가입 페이지와 배너 카드의 레이아웃 클래스가 변경되었습니다. h-full 제거와 flex-1 적용으로 높이/성장 방식이 조정되었고, 배너 버튼에 min-h-[9.6rem]가 추가되었습니다. 로직, 상태, 이벤트 핸들러 변경은 없습니다.

Changes

Cohort / File(s) Summary of changes
Sign-up 레이아웃 높이 조정
src/pages/sign-up/sign-up.tsx, src/pages/sign-up/components/signup-step.tsx
컨테이너에서 h-full 제거, 내부 래퍼에 flex-1 추가로 높이/플렉스 확장 방식 변경
배너 카드 버튼 최소 높이 추가
src/shared/components/card/banner-card/banner-card.tsx
버튼에 min-h-[9.6rem] 추가, JSX 줄바꿈/포맷팅 변경(동작 동일)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Assessment against linked issues

Objective Addressed Explanation
회원가입/홈 디자인 수정 (#388)

Assessment against linked issues: Out-of-scope changes

(없음)

Possibly related PRs

Suggested labels

fix, 희선

Suggested reviewers

  • Dubabbi
  • bongtta
  • yeeeww

Poem

토끼는 폴짝, div는 늘쭉—flex-1로 쑥쑥 자라나고,
h-full은 안녕, 여백은 살짝—레이아웃 고요히 가지런히.
배너 버튼은 키를 컸지, min-h로 빳빳하게.
오늘도 CSS 한 줌으로, 화면은 말끔해졌지.
(\*^▿^)/ 🥕


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e5ff4e3 and f2347b0.

📒 Files selected for processing (3)
  • src/pages/sign-up/components/signup-step.tsx (1 hunks)
  • src/pages/sign-up/sign-up.tsx (1 hunks)
  • src/shared/components/card/banner-card/banner-card.tsx (1 hunks)
✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#388/hotfix-design

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 and usage tips.

@github-actions github-actions bot added the 희선 label Sep 8, 2025
@heesunee heesunee added the fix label Sep 8, 2025
@heesunee heesunee merged commit df14d33 into develop Sep 8, 2025
5 of 7 checks passed
@heesunee heesunee deleted the fix/#388/hotfix-design branch September 8, 2025 07:36
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

MATEBALL-STORYBOOK
⚾ Storybook 배포가 완료되었습니다!

@heesunee heesunee restored the fix/#388/hotfix-design branch September 8, 2025 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 회원가입, 홈 디자인 수정합니다.

1 participant