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] #4 : 캔버스에 선그리기, 이동, 확대/축소 훅 #205

Merged
merged 9 commits into from
Nov 19, 2024
Merged

[FE][Feat] #4 : 캔버스에 선그리기, 이동, 확대/축소 훅 #205

merged 9 commits into from
Nov 19, 2024

Conversation

juwon5272
Copy link
Collaborator

@juwon5272 juwon5272 commented Nov 19, 2024 •
edited
Loading

📝 PR 개요

  • 캔버스에 전 찍어서 선 그리기
    • 점이 1개일 경우 점만 찍힘
    • 점이 2개 이상일 경우 선으로 이어짐
  • 마우스를 눌러서 캔버스 내에 그림들 이동 가능
  • 마우스 휠을 이용하여 캔버스 확대 / 축소 구현
  • undo redo 기능

✅ 체크리스트 (Checklist)

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

🔄 관련 이슈 (Linked Issues)

#4 사용자가 지도에 자유롭게 선을 그을 수 있다
#201 undo 버튼을 누른다면 그렸던 최신 선의 하나가 지워진다
#202 redo 버튼을 누른다면 undo 했던 선의 하나가 복구된다

📷 스크린샷 및 동영상

2024-11-19.2.13.26.mov

🧪 테스트 방법

  • 페이지에서 <Linedrawer />

@juwon5272
[FE][Feat] #4 : 캔버스에 선그리기 훅
e5eb0f4
@juwon5272
[FE][Feat] #4 : 캔버스에서 panning 훅
71c8928
@juwon5272
[FE][Feat] #4 : 캔버스에 마우스 휠로 확대 및 축소 훅
61b5718
@juwon5272
[FE][Feat] #4 : 분리한 훅에 맞게 컴포넌트 변경
acfc1b4
@juwon5272
[FE][Feat] #4 : 찍은 점이 1개일 경우 점이 찍히게 구현
2ae0593
@juwon5272
[FE][Feat] #201 : undo redo 버튼 구현
b8ca237 
- undo 버튼을 클릭하면 최근에 그린 선이 1개 지워집니다
- redo 버튼을 클릭하면 undo한 선중에 최신의 선이 복구됩니다
- undo나 redo할게 없을 경우에는 disabled 상태가 되고 반투명하게 보여집니다
@juwon5272
[FE][Refactor] #4 : 확대/축소 상수 변수화
6667627
@juwon5272
[FE][Refactor] #4 : 네이버지도 확대/축소 배열 변수화
13e12d8
@juwon5272
[FE][Refactor] #4 : Linedrawer 매개변수 수정
1f079a2
@github-actions github-actions bot added the 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) label Nov 19, 2024
happyhyep
happyhyep approved these changes Nov 19, 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

@juwon5272 juwon5272 added 작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경) and removed 확인 요청 리뷰어에게 리뷰 요청 PR 날린 상태 (PR 머지 전) labels Nov 19, 2024
@juwon5272 juwon5272 merged commit 70ba192 into frontend Nov 19, 2024
3 of 6 checks passed
@effozen effozen deleted the feature/fe/#4-drawroute branch December 5, 2024 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@effozen effozen effozen approved these changes

@happyhyep happyhyep happyhyep approved these changes

Assignees

@juwon5272 juwon5272

Labels
작업 완료 PR이 제대로 되어 approve 된 상태 (PR 받은 두명 중 마지막 사람이 상태 변경)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juwon5272 @happyhyep @effozen