14-6 [FE] [논문 상세 - 네트워크 차트] reference 목록의 논문이나 노드를 hover 시 해당하는 논문정보를 강조해주는 기능을 추가한다.

boostcampwm-2022 / web18-PRV

논문 인용관계를 밤하늘의 별 처럼 표시해보자 🌟-🌟

https://paperef.com

143 stars 3 forks source link

Closed Palwol closed 1 year ago

Palwol commented 1 year ago

그래프 hover

기존에는 그래프 zoom이 무한으로 가능했는데, zoom에 제한이 있는 것이 사용자 경험 상 더 좋을 것이라고 생각해서 최소, 최대 제한을 걸어봤습니다. 사용해보시고 기존이 나을지, 제한이 걸려있는게 나을지 의견 부탁드립니다.
ReferenceGraph 컴포넌트의 코드들을 custom hook으로 분리하면서 hooks 폴더가 방대해지고, 여러 컴포넌트에서 사용하는 hook들이 한 폴더에 모여있어서 혼란스럽다는 생각이 들었습니다. 따라서 hooks 폴더를 도메인(page)별로 분리하면 어떨까 하는데 다른 분들의 의견이 궁금합니다!

leesungbin commented 1 year ago

저는 zoom 제한 둔게 더 좋아요~ 우리 서비스가 아름답게 보일 수 있는 범위 안에서 사용자가 보면 좋을 것 같다는 생각에서 그렇습니다.
개인적으로는 hook들이 page별로 분리되어있으면 오히려 코드를 이해하기 어려운 것 같아요,, 현재 상황에서는 d3와 관련되어 있는지 여부로 hooks 폴더 내부에서 분리하는게 나을 것 같다는 생각이 듭니다.
any type의 영향력이 점점 커지는 것 같습니다.
로컬 테스트 완료

Palwol commented 1 year ago

저는 zoom 제한 둔게 더 좋아요~ 우리 서비스가 아름답게 보일 수 있는 범위 안에서 사용자가 보면 좋을 것 같다는 생각에서 그렇습니다.

피드백 감사합니다~ 👍

개인적으로는 hook들이 page별로 분리되어있으면 오히려 코드를 이해하기 어려운 것 같아요,, 현재 상황에서는 d3와 관련되어 있는지 여부로 hooks 폴더 내부에서 분리하는게 나을 것 같다는 생각이 듭니다.

hooks 폴더 내부에서 폴더를 분리하는 것도 좋을 것 같네요! 좋은 의견 감사합니다.

any type의 영향력이 점점 커지는 것 같습니다.

d3에서 nodes와 links 데이터의 타입을 any로 관리하고 있다보니 이 파장이 여기저기 퍼지고 있습니다.. d3에서 계산하면서 중간에 데이터 타입을 바꿔버려서 static한 타입을 찾는 데 애를 먹고 있습니다. 타입을 찾는데로 수정하도록 하겠습니다!

Palwol commented 1 year ago