Skip to content

Conversation

@djk01281
Copy link
Collaborator

@djk01281 djk01281 commented Nov 19, 2024

🔖 연관된 이슈

📂 작업 내용

  • 움직일 때, 가장 가까운 엣지로 변경
  • 가장 가까운 핸들에 연결
  • 노드 간 1개의 엣지만 가능하게 수정

📑 참고 자료 & 스크린샷 (선택)

edge-change-demo.mp4

📢 리뷰 요구사항 (선택)

  • 기본적으로 data-id가 달려있더라구요..! 그래서 거기서 getBoundingClientRect()을 했습니다. 노드 width, height를 그대로 쓰는 것도 생각해봤는데 나중에 노드 크기가 다 다르면 안될 것 같아서요. 사실 ref로 가져올 수 있으면 제일 좋긴 할텐데 방법을 못찾아서..
  • 로직 자체가 너무 코딩테스트 같아서.. 리팩토링을 해야되긴 할텐데.. 이게 ydoc이랑 얽혀 있다보니까 hook으로 분리하는게 쉽지 않네요..

@djk01281 djk01281 added 🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우 labels Nov 19, 2024
@djk01281 djk01281 requested a review from yewonJin November 19, 2024 05:24
@djk01281 djk01281 self-assigned this Nov 19, 2024
@djk01281 djk01281 marked this pull request as draft November 19, 2024 05:27
@djk01281 djk01281 marked this pull request as ready for review November 19, 2024 05:29
Copy link
Collaborator

@yewonJin yewonJin left a comment

Choose a reason for hiding this comment

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

P5: 고생하셨습니다!!!

@yewonJin yewonJin merged commit eda692a into develop Nov 19, 2024
21 checks passed
@yewonJin yewonJin deleted the feature-fe-#200 branch November 19, 2024 06:11
@Tolerblanc Tolerblanc restored the feature-fe-#200 branch November 20, 2024 02:58
@Tolerblanc Tolerblanc deleted the feature-fe-#200 branch November 25, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

엣지 연결 로직 수정

3 participants