Skip to content

develp branch 작업내용 머지 : develop -> main #266

Merged
ho0010 merged 3 commits intomainfrom
develop
Sep 26, 2025
Merged

develp branch 작업내용 머지 : develop -> main #266
ho0010 merged 3 commits intomainfrom
develop

Conversation

@ho0010
Copy link
Collaborator

@ho0010 ho0010 commented Sep 26, 2025

Summary by CodeRabbit

  • Style
    • 모달 열림 시 배경 페이지 스크롤이 잠기도록 개선하여 의도치 않은 스크롤 이동을 방지했습니다.
    • 모달 콘텐츠 하단 여백을 확대했습니다: CompletedGroupModal 70px, MyGroupModal 60px로 시각적 안정성과 터치 영역을 개선했습니다.
    • 모달 콘텐츠 영역의 스크롤바를 브라우저별로 숨겨 더 깔끔한 화면을 제공합니다.
    • 불필요한 주석을 정리해 가독성을 향상했습니다.

밖의 영역 스크롤 되는 버그 수정
모달 내부 content 스크롤 숨김 처리
content 가려짐 방지를 위한 margin 추가
content 가려짐 방지를 위한 margin 추가
fix: 완료된 모임방, 내 모임방 content 가려지는 버그
@ho0010 ho0010 self-assigned this Sep 26, 2025
@ho0010 ho0010 added 🐞 BugFix Something isn't working 🌏 Deploy 배포 관련 labels Sep 26, 2025
@vercel
Copy link

vercel bot commented Sep 26, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
thip Ready Ready Preview Comment Sep 26, 2025 0:23am

@ho0010 ho0010 merged commit 584a6d9 into main Sep 26, 2025
1 check passed
@coderabbitai
Copy link

coderabbitai bot commented Sep 26, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

  • CompletedGroupModal: 모달 마운트 시 문서 바디 스크롤을 비활성화하고 언마운트 시 복원하는 useEffect 추가. Content 스타일에 하단 마진 70px과 스크롤바 숨김 CSS 추가. 주석 제거.
  • MyGroupModal: Content 스타일에 하단 마진 60px 추가.

Changes

Cohort / File(s) 요약
모달 스크롤 잠금 및 스타일 조정
src/components/group/CompletedGroupModal.tsx
모달 생명주기에 맞춰 document.body 스크롤 잠금/복원 useEffect 추가. Content에 margin-bottom: 70px 및 브라우저별 스크롤바 숨김 CSS 적용. 불필요 주석 제거.
모달 하단 여백 추가
src/components/group/MyGroupModal.tsx
Content에 margin-bottom: 60px 추가. 기타 동작 변화 없음.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User as 사용자
  participant UI as 페이지
  participant Modal as CompletedGroupModal
  participant Body as document.body

  User->>UI: 모달 열기
  UI->>Modal: 마운트
  activate Modal
  Modal->>Body: overflow: hidden 설정 (스크롤 잠금)
  note right of Modal: Content: 하단 여백, 스크롤바 숨김 스타일

  User->>UI: 모달 닫기
  UI-->>Modal: 언마운트
  Modal->>Body: overflow 원복 (스크롤 복원)
  deactivate Modal
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

🎨 Html&css

Poem

(\(\
( -.-) 모달이 둥실, 스크롤은 멈칫—
o_(")(") 살금살금 여백을 깔고
완료 방엔 살포시 숨긴 스크롤,
나의 방엔 넉넉한 바닥 공기.
탁— 닫히면 원래대로, 깡충!

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 0224565 and b26c58a.

📒 Files selected for processing (2)
  • src/components/group/CompletedGroupModal.tsx (2 hunks)
  • src/components/group/MyGroupModal.tsx (1 hunks)

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix Something isn't working 🌏 Deploy 배포 관련

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant