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

[FE][Feat] #184 : Canvas와 Map 레이어 구분 및 겹쳐서 보이게 구현 #198

Merged
merged 9 commits into from
Nov 19, 2024
Merged

[FE][Feat] #184 : Canvas와 Map 레이어 구분 및 겹쳐서 보이게 구현 #198

merged 9 commits into from
Nov 19, 2024

Conversation

effozen
Copy link
Collaborator

@effozen effozen commented Nov 18, 2024

📝 PR 개요

Canvas와 Map 레이어 구분 및 겹쳐서 보이게 구현

🔍 변경 사항

  • Canvas 컴포넌트 추가
  • Layout, Header 등 기존에 레이아웃 요소의 z-index 수정
  • Canvas와 Map 합쳐서 하나의 컴포넌트로 사용 가능한 CanvasWithMap 컴포넌트 구현

✅ 체크리스트 (Checklist)

  • 코드가 빌드 오류 없이 잘 작동하는지 확인
  • 테스트가 통과하는지 확인
  • 스타일 가이드와 일관성을 유지했는지 확인
  • 관련 문서가 업데이트되었는지 확인 (선택 사항)
  • 리뷰어가 이해할 수 있도록 주석이나 설명을 추가했는지 확인

🔄 관련 이슈 (Linked Issues)

#184

📷 스크린샷 및 동영상 (선택 사항)

캔버스 컴포넌트 위에 무언가를 그리지 않아서 사진 생략

effozen and others added 9 commits November 17, 2024 05:30
@effozen
[ENV][Fix] #1 : auto-assign 라벨 바뀔때마다 되는 문제 수정
b1f6875
@effozen
[ENV][Fix] #1 : auto-assign 라벨 바뀔때마다 되는 문제 재수정
1baf6e3 
action 코드가 제대로 반영이 안되어있어서 재수정
@effozen
[ENV][Fix] #1 : auto-merge 정상적으로 동작안되는 문제 수정
f17994a
@effozen
Merge pull request #180 from boostcampwm-2024/feature/env/#1-auto-mer…
a7e2d1f 
…ge-bug-fix

[ENV][Fix] #1 : auto-merge 정상적으로 동작안되는 문제 수정
@effozen
Merge pull request #179 from boostcampwm-2024/feature/env/#1-auto-ass…
fe41f0c 
…ign-bug-fix-1

[ENV][Fix] #1 : auto-assign 라벨 바뀔때마다 되는 문제 수정
@effozen
[FE] : main branch merge
7649e8f
@effozen
[FE][Style] #184 : 헤더 z-index 수정
efe3fb8
@effozen
[FE][Style] #184 : canvas랑 지도 겹쳐서 보이는 레이아웃 구현
8275109
@effozen
[ENV][test] : 프로젝트 전체에 테스트 코드 없을 떄 CI 오류나는 것 방지용 더미 테스트 도구
164fecd
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 18, 2024
happyhyep
happyhyep approved these changes Nov 18, 2024
View reviewed changes
Copy link
Member

@happyhyep happyhyep left a comment

Choose a reason for hiding this comment

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

LGTM

@effozen effozen linked an issue Nov 18, 2024 that may be closed by this pull request
지도와 캔버스를 별도의 레이어로 구분하고, 한 화면 위에 출력한다. #184
Closed
3 tasks
leedongyull
leedongyull approved these changes Nov 19, 2024
View reviewed changes
Copy link
Collaborator

@leedongyull leedongyull left a comment

Choose a reason for hiding this comment

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

LGTM

@effozen effozen merged commit 82baf16 into frontend Nov 19, 2024
3 of 6 checks passed
@effozen effozen deleted the feature/fe/#184-map-and-canvas branch December 5, 2024 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@happyhyep happyhyep happyhyep approved these changes

@leedongyull leedongyull leedongyull approved these changes

Assignees

@effozen effozen

Labels
확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

지도와 캔버스를 별도의 레이어로 구분하고, 한 화면 위에 출력한다.
3 participants
@effozen @leedongyull @happyhyep