boostcampwm-2022 / Web35-CryptoGraph

실시간 암호화폐 데이터 시각화 웹 서비스
19 stars 4 forks source link

Issue#8/candle chart represent data #34

Closed kongyb closed 1 year ago

kongyb commented 1 year ago

개요

작업사항

생각해볼점

이미지

Nov-24-2022 04-06-08

baldwinIV commented 1 year ago

매직넘버 상수화(UI의 font-size) 반응형고려하면 어떻게 해야할까

ref를 argument로 넣고, ref의 width, height를 이벤트 리스너로 걸어서 변할떄마다 상태를 내밷는 useRefSize라는 커스텀훅을 하나 만들어서 사용하면 될 것 같습니다.

baldwinIV commented 1 year ago

격자와 차트가 틀어지는 문제도 차트의 이동과 다시계산되는 xScale의 싱크가 맞지 않는 문제라고 추측하고 있는데, xAxis와 chart-area를 합쳐서 같이 translate시켜보면 어떨지..??

baldwinIV commented 1 year ago

updateChart함수는 단순히 매개변수를 이용하여 차트를 적절하게 렌더링하는 역할을 하고있는데, 차트를 렌더링하기 전에 매개변수의 option의 인덱스 정보를 활용하여 현재 갖고있는 데이터가 렌더링되야하는 데이터를 포함하는지 확인하고 포함하지 않는다면 데이터를 fetch해오는 방향으로 핸들링 하는 방법 생각해보기.

잘 이해가 되지 않는데, 다음 기회에, 스크럼때 한번 설명해주시면 감사하겠습니다.