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.

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

노드 생성 드래그 관련 버그 개선 #169

Merged
merged 11 commits into from
Nov 28, 2024
Merged

Conversation

CatyJazzy
Copy link
Collaborator

✏️ 한 줄 설명

이 PR의 주요 변경 사항이나 구현된 내용을 간략히 설명해 주세요.

드래그 상태를 변경 관련한 성능 개선, 드롭 후 팔레트 메뉴 표시되었을 때 숨김처리하는 로직을 개선했어요.

✅ 작업 내용

  • 팔레트 메뉴 표시 후 바깥 영역 클릭 시 숨김 처리
  • 드래그 상태 업데이트에 requestAnimationFrame 적용

🏷️ 관련 이슈

📸 스크린샷/영상

이번 PR에서 변경되거나 추가된 뷰가 있는 경우 이미지나 동작 영상을 첨부해 주세요.

화면 기록 2024-11-28 오후 7 41 03

📌 리뷰 진행 시 참고 사항

리뷰 코멘트 작성 시 특정 사실에 대해 짚는 것이 아니라 코드에 대한 의견을 제안할 경우, 강도를 함께 제시해주세요! (1점: 가볍게 참고해봐도 좋을듯 ↔ 5점: 꼭 바꾸는 게 좋을 것 같음!)

  • [🚨5점] 윈도우에서 여전히 노드 위의 우클릭 시 컨텍스트 메뉴가 안 뜨는지 확인부탁드립니다!
  • radix-dialog를 사용하려고 했으나 조금 복잡해져서 stage의 onClick핸들러에서 처리했어요. 혹시 이렇게 했을 때의 잠재적인 이슈?가 생각난다면 알려주세요.

@CatyJazzy CatyJazzy added this to the Node 생성 milestone Nov 28, 2024
@CatyJazzy CatyJazzy self-assigned this Nov 28, 2024
@CatyJazzy CatyJazzy requested a review from a team as a code owner November 28, 2024 11:06
@CatyJazzy CatyJazzy requested review from hoqn and heegenie and removed request for a team November 28, 2024 11:06
@hoqn
Copy link
Collaborator

hoqn commented Nov 28, 2024

alert으로 추가하셨군요..! 제가 앞선 PR에서 지원되지 않는 메뉴는 없애놨었는데 다시 복구해도 좋아요! alert으로 하는 게 더 나을 것 같아요
(메뉴가 적으니 육각형 메뉴가 너무 휑 하더라고요..ㅎㅎ)

@hoqn
Copy link
Collaborator

hoqn commented Nov 28, 2024

저 오른쪽 클릭이 아예 안 되고, 일반 클릭으로 먹어요..! (맥북, 노드 엣지 모두)

@CatyJazzy
Copy link
Collaborator Author

일단 우클릭은 다시 작동하도록 복귀했고 이벤트 핸들러 넘겨주는 것을 stage > node, edge로 변경했어요. 윈도우 테스트까지 해봐야 ... 목표 달성일 것 같습니다. alert는 그대로 살렸습니다. 이번 PR 정말 감사했습니다ㅠㅠ (@hoqn ✨)

@CatyJazzy CatyJazzy merged commit 772a5ce into dev Nov 28, 2024
@CatyJazzy CatyJazzy deleted the feature/improve-dragUpdate branch December 4, 2024 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants