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

인터랙션 가이드 UI 추가 #197

Merged
merged 2 commits into from
Dec 2, 2024
Merged

인터랙션 가이드 UI 추가 #197

merged 2 commits into from
Dec 2, 2024

Conversation

heegenie
Copy link
Collaborator

@heegenie heegenie commented Dec 2, 2024

✏️ 한 줄 설명

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

사용자 편의를 위해 마우스 hover 시 인터랙션 가이드를 확인할 수 있는 카드를 추가했습니다.

✅ 작업 내용

  • 인터랙션 가이드 컴포넌트 구현

🏷️ 관련 이슈

📸 스크린샷/영상

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

Screen Recording 2024-12-02 at 7 32 22 PM

📌 리뷰 진행 시 참고 사항

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

  • 더 추가되어야 하는 인터랙션이나 수정해야 할 문구가 있으면 자유롭게 남겨주세요!
  • 헤더 높이 offset 계산해서 위치 잡도록 구현하려 했으나, offset 값이 계속 undefined로 떠서 일단 헤더 높이값 +a 로 구현된 상태입니다.. 이건 문서화 작업 후 시간이 남으면 다시 수정해보겠습니다!

@heegenie heegenie added this to the UX 개선 milestone Dec 2, 2024
@heegenie heegenie self-assigned this Dec 2, 2024
@heegenie heegenie requested a review from a team as a code owner December 2, 2024 10:36
@heegenie heegenie requested review from parkblo and CatyJazzy and removed request for a team December 2, 2024 10:36
@heegenie
Copy link
Collaborator Author

heegenie commented Dec 2, 2024

Screenshot 2024-12-02 at 7 42 15 PM

지후님이 알려 주신 내용 추가해서 이렇게 수정했습니다!

Copy link
Collaborator

@CatyJazzy CatyJazzy left a comment

Choose a reason for hiding this comment

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

(1점) 모바일에서도 볼 수 있는 방법을 추가할 수 있을까요 ㅎㅎ
(태스크 작업이 커지면 우선이 될 필요는 없다고 생각합니다!)

Copy link
Collaborator

@parkblo parkblo left a comment

Choose a reason for hiding this comment

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

희진님 고생 많으셨습니다 👍
모바일에서는 만들어주신 가이드를 확인하기 어려웠는데, 찾아보니 hover card는 desktop 만을 고려한 컴포넌트라고 .. 하네요 ㅎㅎ Improvement: Tooltip and HoverCard Mobile Support · Issue #2402 · shadcn-ui/ui
모바일 이외는 정상작동 확인하여 approve 드립니다 !

@heegenie
Copy link
Collaborator Author

heegenie commented Dec 2, 2024

앗 hover card가 모바일에서 동작하지 않는군요... 😭 이 부분은 그럼 모바일 지원 이슈로 따로 생성해서 PR 올리겠습니다!

@heegenie heegenie merged commit 8b7feac into dev Dec 2, 2024
@heegenie heegenie deleted the feature/interaction-guide branch December 2, 2024 14:55
CatyJazzy pushed a commit that referenced this pull request Dec 3, 2024
* feat: 인터랙션 가이드 추가

* chore: 인터랙션 가이드 내용 추가
CatyJazzy pushed a commit that referenced this pull request Dec 3, 2024
* feat: 인터랙션 가이드 추가

* chore: 인터랙션 가이드 내용 추가
fru1tworld added a commit that referenced this pull request Dec 3, 2024
* feat: space-delete-api

* fix: 에러 메시지 추가

* fix: json pasing 안되는 이슈 해결

* refactor: validation Service validation으로 네이밍 변경

* feat: space update 로직 추가

* fix: updateSpaceByName

* fix: 노드 위 우클릭시 이동모드 자동 활성화되는 오류 수정 (#195)

fix: 노드 위 우클릭 시 이동모드 활성화되는 것 방지

* fix: 이동모드 활성화 함수에서 button 판별오류 수정 (#196)

* fix: mouse-event에 대해서만 button 판별하도록 수정

* feat: 인터랙션 가이드 UI 추가 (#197)

* feat: 인터랙션 가이드 추가

* chore: 인터랙션 가이드 내용 추가

* feat: 모바일 환경에서 더보기 버튼을 통한 노드.간선 편집 (#199)

* feat: 터치디바이스 여부 확인 로직

* feat: 모바일환경에서 더보기 버튼 클릭 시 context-menu 표시

* feat: 더보기 버튼 클릭 시 clientX, clientY값 정의하여 전달

* feat: 간선 삭제 버튼 스타일 수정

* feat: 삭제 버튼 터치 핸들러 연결

* feat: 서브스페이스 삭제 api 연동

* feat: 작업 내용 저장

* feat: save

* fix: space delete id가 아니라 src에 있는 걸 갖고 올 수 있도록 수정

* feat: deleteNote Controller API 추가

* feat: 서브스페이스 수정 api 연동

---------

Co-authored-by: CatyJazzy <[email protected]>
Co-authored-by: Heejin Na <[email protected]>
Co-authored-by: CatyJazzy <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

인터랙션에 대한 가이드, 튜토리얼 UI 구성
3 participants