[view] 분석 컴포넌트 관련 논의

[HIITMEMARIO]

안녕하세요! 분석 컴포넌트 팀 회의결과 공유드립니다...! 저희는 우선 기존의 내용을 수정하는 것도 좋지만 핵심적으로 어떤 분석 차트를 추가할것인가를 고민하게 되었습니다...! 아이디어는 두개였습니다.

FEAT

1. 폴더, 파일 별 insertion, deletion 수 나타내기

현재 변경된 파일 수만 알 수 있고 총 얼마의 insertion, deletion이 발생했는지 알 수 없습니다. 따라서
tooltip 형태로 insertion, deletion 띄워주고 폴더 선택 시 해당 폴더의 하위 폴더 / 파일에 대한 insertion, deletion 정보를 아래 사진과 같은 차트 형식으로 보여주게 하려고 합니다.
뿐만 아니라 현재 commit 내역 클릭 시 이벤트가 발생하지 않는데, 클릭 시 해당 커밋에 대한 정보를 차트에 반영하는 기능도 추가하려 합니다.

2. 커밋 컨벤션 레이블을(fix,bug...) 베이스로 데이터를 시각화시키기

현재 차트로 시각화된 데이터에서는 각 사용자별로 파일별 커밋별 데이터는 보여주고 있으나 특정 파일을 어떠한 목적으로 손을 댄건지는 알 수 없습니다. 따라서 현재 프로젝트에서 지키고 있는 커밋 내역 중 커밋 컨벤션(fix,bug,view,engine…)을 베이스로 특정 파일을 건드린 목적을 분명하게 시각화 시키면 좋을 것 같다고 생각했습니다. 아래 형식의 차트로 보여주고 chart bar의 색을 실제 깃헙에서의 레이블 색과 동일하게 준다면 좀 더 직관적일 것 같습니다.

예상되는 문제

1. 커밋 컨벤션을 지키지 않는 사람들의 데이터를 어떻게 처리할 것인가?

Fix

그 외에 수정이 필요해 보이는 부분도 얘기가 나왔습니다. 해당 부분은 다른 팀과 같이 상의해보면 좋을 문제인것 같습니다!^^

1. selection 제대로 적용 안됨
2. 선 구분
3. 글자 겹쳐서 보이는 현상
4. 글자 잘리는 현상

이 외의 의견과 문제점 지적 모두 환영입니다....!!

[bbanderson]

1, 2번 모두 공감하는 내용이었습니다!👍
(저도 특히 커밋을 클릭했을 때 기대되는 이벤트가 없어서 아쉬웠거든요..!)

1. 그러면 중간을 기준으로, Insertion은 오른쪽으로, Deletion은 왼쪽으로 표시되는 걸까요??
2. 그래프가 더 풍부하고 다채로워질 것으로 기대되네요~

[ytaek]

1. 폴더, 파일 별 insertion, deletion 수 나타내기

현재 변경된 파일 수만 알 수 있고 총 얼마의 insertion, deletion이 발생했는지 알 수 없습니다. 따라서 tooltip 형태로 insertion, deletion 띄워주고 폴더 선택 시 해당 폴더의 하위 폴더 / 파일에 대한 insertion, deletion 정보를 아래 사진과 같은 차트 형식으로 보여주게 하려고 합니다. 뿐만 아니라 현재 commit 내역 클릭 시 이벤트가 발생하지 않는데, 클릭 시 해당 커밋에 대한 정보를 차트에 반영하는 기능도 추가하려 합니다.

차트를 새로 만든다는 것인지, 어떤 부분에 click이벤트를 넣는 것인지 등 등
말씀하신 부분들의 디테일한 부분들을 잘 정리해서 나열해놓은뒤,
하나하나씩 이슈로 만들어놓고 진행하시는 걸 추천드려요~.

요 그림은 참고만 하실 것 같긴한데, 실제 적용하려면 아마 많은 부분 수정이 필요할꺼에요 😸
초안 완성되면 저도 같이 리뷰하도록 하겠습니다!!

2. 커밋 컨벤션 레이블을(fix,bug...) 베이스로 데이터를 시각화시키기

현재 차트로 시각화된 데이터에서는 각 사용자별로 파일별 커밋별 데이터는 보여주고 있으나 특정 파일을 어떠한 목적으로 손을 댄건지는 알 수 없습니다. 따라서 현재 프로젝트에서 지키고 있는 커밋 내역 중 커밋 컨벤션(fix,bug,view,engine…)을 베이스로 특정 파일을 건드린 목적을 분명하게 시각화 시키면 좋을 것 같다고 생각했습니다. 아래 형식의 차트로 보여주고 chart bar의 색을 실제 깃헙에서의 레이블 색과 동일하게 준다면 좀 더 직관적일 것 같습니다.

예상되는 문제

1. 커밋 컨벤션을 지키지 않는 사람들의 데이터를 어떻게 처리할 것인가?

원래 엔진쪽에서 commit type을 분류해줘야 하는데 현재는 그런게 없죠 ㅎㅎ
말씀하신 것 처럼 commit convention을 쓴 경우에 한해서 위의 것들을 해보는 건 좋은 시도 같습니다.

bar chart의 색상 같은 경우는 전체 컬러 시스템을 다루는 다른 모듈과 논의를 같이 해보면 좋을 것 같구요.
실제 github의 색상과 연동하는 건 쉽지 않을 것 같긴 합니다.

전부 중요한 내용을 다루신 것 같네요!!
향후 새 feature들의 모습들이 기대됩니다 😃😃😃😃😃😃

[seungineer]

다뤄 주신 내용 모두 githru에 포함되면 좋을 것 같네요. 👍👍 분석 컴포넌트 팀 화이팅 ~! 🔥🔥🔥

Fix

그 외에 수정이 필요해 보이는 부분도 얘기가 나왔습니다. 해당 부분은 다른 팀과 같이 상의해보면 좋을 문제인것 같습니다!^^

1. selection 제대로 적용 안됨
2. 선 구분
3. 글자 겹쳐서 보이는 현상
4. 글자 잘리는 현상

Folder tree는 2번, 3번 해결이 시급하다고 생각합니다!
왜냐하면 2번으로 인해 어떤 역할을 하는 컴포넌트인지 한 눈에 와닿지 않는 것 같습니다. 그리고 3번으로 인해 folder tree 컴포넌트의 퀄리티가 떨어져 보이구요. 🥲

추후 다 같이 얘기해볼 수 있는 기회가 있으면 좋겠네요 ! ^_^

[hhbb0081]

1. 그러면 중간을 기준으로, Insertion은 오른쪽으로, Deletion은 왼쪽으로 표시되는 걸까요??

네 제가 생각한 방식은 그렇습니다~! insertion은 파란색으로 오른쪽, deletion은 빨간색으로 왼쪽으로 표시하려고 생각 중입니다!
질문 감사합니다😊

[hhbb0081]

1. 폴더, 파일 별 insertion, deletion 수 나타내기

현재 변경된 파일 수만 알 수 있고 총 얼마의 insertion, deletion이 발생했는지 알 수 없습니다. 따라서 tooltip 형태로 insertion, deletion 띄워주고 폴더 선택 시 해당 폴더의 하위 폴더 / 파일에 대한 insertion, deletion 정보를 아래 사진과 같은 차트 형식으로 보여주게 하려고 합니다. 뿐만 아니라 현재 commit 내역 클릭 시 이벤트가 발생하지 않는데, 클릭 시 해당 커밋에 대한 정보를 차트에 반영하는 기능도 추가하려 합니다.

차트를 새로 만든다는 것인지, 어떤 부분에 click이벤트를 넣는 것인지 등 등 말씀하신 부분들의 디테일한 부분들을 잘 정리해서 나열해놓은뒤, 하나하나씩 이슈로 만들어놓고 진행하시는 걸 추천드려요~.

요 그림은 참고만 하실 것 같긴한데, 실제 적용하려면 아마 많은 부분 수정이 필요할꺼에요 😸 초안 완성되면 저도 같이 리뷰하도록 하겠습니다!!

넵! 피드백 반영하여 잘 쪼개서 구체화시킨 후 이슈 올리도록 하겠습니다!
좋은 지적 감사합니다 😊

[hhbb0081]

다뤄 주신 내용 모두 githru에 포함되면 좋을 것 같네요. 👍👍 분석 컴포넌트 팀 화이팅 ~! 🔥🔥🔥

Fix

그 외에 수정이 필요해 보이는 부분도 얘기가 나왔습니다. 해당 부분은 다른 팀과 같이 상의해보면 좋을 문제인것 같습니다!^^

1. selection 제대로 적용 안됨
2. 선 구분
3. 글자 겹쳐서 보이는 현상
4. 글자 잘리는 현상

Folder tree는 2번, 3번 해결이 시급하다고 생각합니다! 왜냐하면 2번으로 인해 어떤 역할을 하는 컴포넌트인지 한 눈에 와닿지 않는 것 같습니다. 그리고 3번으로 인해 folder tree 컴포넌트의 퀄리티가 떨어져 보이구요. 🥲

추후 다 같이 얘기해볼 수 있는 기회가 있으면 좋겠네요 ! ^_^

말씀대로 Fix issue도 우선순위 부여하여 차례로 구현해나가면 좋을 것 같습니다!
의견 감사합니다 😊

[pcwadarong]

Fix

그 외에 수정이 필요해 보이는 부분도 얘기가 나왔습니다. 해당 부분은 다른 팀과 같이 상의해보면 좋을 문제인것 같습니다!^^

1. selection 제대로 적용 안됨
2. 선 구분
3. 글자 겹쳐서 보이는 현상
4. 글자 잘리는 현상

해당 부분 관련해서 이슈 올렸었는데 참고하시면 좋을 것 같습니다~
#529 #530 #531

컬러 시스템의 경우 현재는 3가지의 각기 다른 컬러로 잡았지만 그래프 ui에 따라 가짓수와 색감 등등 전적으로 조정 가능합니다!