Skip to content

[#93] Refactor/공통 커스텀스크롤바#97

Merged
LMS10 merged 4 commits intomainfrom
refactor/공통_커스텀스크롤바
Feb 17, 2025

Hidden character warning

The head ref may contain hidden characters: "refactor/\uacf5\ud1b5_\ucee4\uc2a4\ud140\uc2a4\ud06c\ub864\ubc14"
Merged

[#93] Refactor/공통 커스텀스크롤바#97
LMS10 merged 4 commits intomainfrom
refactor/공통_커스텀스크롤바

Conversation

@LMS10
Copy link
Owner

@LMS10 LMS10 commented Feb 16, 2025

#️⃣ Issue

📝 Description

커스텀 스크롤바 스타일 추가, 모달 외부 스크롤 방지 및 CLS 해결 작업에 관한 PR입니다.

  • custom-scrollbar 클래스를 통해 스크롤바 디자인 커스텀
    • @layer base에서 커스텀 스크롤 전역 적용
    • 모달은 모달 내부로 별도로 감싸서 적용
  • 모달 상태 변경 시 스크롤 락을 관리하고, 스크롤바 CLS 문제를 해결하는 useScrollLock 훅 구현
스크롤

🌀 PR Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항 (오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

✅ Checklist

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • Branch Convention 확인

    feat/ 피쳐, fix/ 버그 수정, refactor/ 개선

  • Base Branch 확인
  • 커밋 메시지 컨벤션 준수
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test Checklist

  • 로컬 작동 확인

Additional Notes

  • (없음)

@LMS10 LMS10 added Priority: Medium 🔖 우선순위 보통 Type: Improve UX ⬆️ UX 개선 labels Feb 16, 2025
@LMS10 LMS10 self-assigned this Feb 16, 2025
@vercel
Copy link

vercel bot commented Feb 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wine ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 16, 2025 6:19pm

@LMS10 LMS10 requested a review from qpalkim February 17, 2025 01:01
@LMS10 LMS10 merged commit e184eb0 into main Feb 17, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor : [공통] 커스텀 스크롤바

2 participants