depth에 따른 노드 색 구현, 불필요한 코드 제거

[yang1318]

관련 이슈

close #7

작업한 내용

• depth에 따른 노드 색 구현
  
  
• 불필요한 코드 제거
  • Node 속성에서 colorRBB 제거
  • NodeView의 drawCircleNode 함수에서 쓰지 않는 매개변수인 depth 제거
  • forEach문을 돌기전에 isNotEmpty를 검사하는 코드 제거 (어차피 empty면 forEach문을 돌지 않고 벗어나니까)

리뷰 후 수정한 부분

• Before / After

[jaehan4707]

필요하지 않는 속성 제거하신거 좋은거 같아요.
그리고 컬러도 값을 하드코딩 한게 아닌 color.xml에 들고온것도 좋은거 같네용.
노드의 깊이마다 색깔이 구분하는건 좋은데 색깔을 조금 더 확실한걸로 수정하면 깊이를 알아보는데 좋을거 같아요

[yang1318]

필요하지 않는 속성 제거하신거 좋은거 같아요. 그리고 컬러도 값을 하드코딩 한게 아닌 color.xml에 들고온것도 좋은거 같네용. 노드의 깊이마다 색깔이 구분하는건 좋은데 색깔을 조금 더 확실한걸로 수정하면 깊이를 알아보는데 좋을거 같아요

감사합니당ㅎㅎ
확실한거면 어떤건가용?? 각 색깔을 변수에 할당해서 이름을 주자는 건가용??

[jaehan4707]

필요하지 않는 속성 제거하신거 좋은거 같아요. 그리고 컬러도 값을 하드코딩 한게 아닌 color.xml에 들고온것도 좋은거 같네용. 노드의 깊이마다 색깔이 구분하는건 좋은데 색깔을 조금 더 확실한걸로 수정하면 깊이를 알아보는데 좋을거 같아요

감사합니당ㅎㅎ 확실한거면 어떤건가용?? 각 색깔을 변수에 할당해서 이름을 주자는 건가용??

음.. 자세한 생각은 안해봣는데 저희 테마에서 사용되는 색깔이 조금은 연하다고 생각이 들어서 조금 더 채도가 진한 색깔이라고 해야하나요?
1층과 3층, 2층과 4층 색깔이 비슷하다고 느껴져서요 아마 밝기나 이런게 변하면 조금 더 느껴지지 않을까..?라는 예상이긴 해요

[yang1318]

음.. 자세한 생각은 안해봣는데 저희 테마에서 사용되는 색깔이 조금은 연하다고 생각이 들어서 조금 더 채도가 진한 색깔이라고 해야하나요? 1층과 3층, 2층과 4층 색깔이 비슷하다고 느껴져서요 아마 밝기나 이런게 변하면 조금 더 느껴지지 않을까..?라는 예상이긴 해요

반영해서 색 좀 더 예쁘게 바꿔봤습니다!