Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Fix] Next.js HMR 스타일 초기화 문제 해결 #103

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

[Fix] Next.js HMR 스타일 초기화 문제 해결 #103

wants to merge 10 commits into from
+208 −169

Conversation

red-dev-Mark
Copy link
Collaborator

@red-dev-Mark red-dev-Mark commented Jan 27, 2025 •
edited
Loading

🚀 풀 리퀘스트 제안

📋 작업 내용

HMR 스타일 초기화 문제 해결

  • dynamic import와 HMR 충돌이 원인
  • 페이지 전체 → 지도 컴포넌트만 dynamic import 적용
  • 스타일 안정성 확보

Modal 포탈 임시 비활성화

  • 빌드 시 document 객체 접근 문제 해결 필요

🔧 변경 사항

  • 💡 비즈니스 로직 (src/, app/ 등)
  • 📦 의존성 (package.json)
  • 📃 문서 (README.md 등)
  • 🔥 삭제된 파일/코드
  • ⚙️ 설정 파일 (.env, .gitignore 등)

📄 기타

  • 지도 컴포넌트와 동일 페이지에 있는 Input 입력 성능 최적화 예정
  • Modal 포탈 구현 개선

@red-dev-Mark red-dev-Mark added the 🐛 Fix 버그 수정 작업 label Jan 27, 2025
@red-dev-Mark red-dev-Mark self-assigned this Jan 27, 2025
@red-dev-Mark red-dev-Mark linked an issue Jan 27, 2025 that may be closed by this pull request
[Fix] Next.js 개발 환경 성능 및 스타일 로딩 이슈 #101
Open
@nanafromjeju nanafromjeju added this to the 버그 수정 milestone Jan 27, 2025
nanafromjeju
nanafromjeju previously approved these changes Jan 27, 2025
View reviewed changes
Copy link
Collaborator

@nanafromjeju nanafromjeju left a comment

Choose a reason for hiding this comment

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

굿굿

@red-dev-Mark
Merge branch 'develop' of https://github.com/Kernel360/KEDV3_monicar_FE…
92f4495 
… into fix/hmr-css-101
@red-dev-Mark
refactor(map): memo를 통한 Map 컴포넌트 리렌더링 최적화 (#101)
c7a02dc
@red-dev-Mark
fix: MarkerClusterer 제거로 맵 스크롤 및 입력 성능 개선 (#101)
151318b 
- 맵 스크롤 시 발생하던 이벤트 블로킹 해결
- 입력 시 발생하던 강제 리플로우 문제 해결
- DOM 측정으로 인한 메인 스레드 블로킹 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nanafromjeju nanafromjeju nanafromjeju approved these changes

Assignees

@red-dev-Mark red-dev-Mark

Labels
🐛 Fix 버그 수정 작업
Projects
None yet
Milestone
버그 수정
Development

Successfully merging this pull request may close these issues.

[Fix] Next.js 개발 환경 성능 및 스타일 로딩 이슈
2 participants
@red-dev-Mark @nanafromjeju