노드를 이동시킬 때 구조 유지 #330
노드를 이동시킬 때 구조 유지 #330
Conversation
hegleB
force-pushed
the
AOS-feat/structural-maintenance
branch
from
30f5b27 to
7ba895a
Compare
|
고생하셨습니다!! |
|
여기에 있는 gif 말씀하시는건가용? |
| val path = | ||
| path.apply { | ||
| reset() | ||
| moveTo(startX, startY) | ||
| cubicTo(midX, startY, midX, endY, endX, endY) | ||
| } |
There was a problem hiding this comment.
이부분을 createPath로 빼니까 더 직관적인 것 같아요! 좋아요
| val linePaint = | ||
| when (context.resources.configuration.uiMode and Configuration.UI_MODE_NIGHT_MASK) { | ||
| Configuration.UI_MODE_NIGHT_YES -> { | ||
| drawInfo.darkModeLinePaint | ||
| } | ||
| val linePaint = getLinePaintForMode() | ||
|
|
||
| Configuration.UI_MODE_NIGHT_NO -> { | ||
| drawInfo.linePaint | ||
| } | ||
| val path = createPath(fromNode, toNode) | ||
| drawPathConditionally(toNode, canvas, path, linePaint) | ||
| } | ||
|
|
||
| else -> drawInfo.linePaint | ||
| } |
There was a problem hiding this comment.
같은 코드라도 함수로 빼서 짧아지니까 더 보기 좋네요😃
| rightLayoutManager.arrangeNode(tree, target as RectangleNode) | ||
|
|
||
| node.children.forEach { nodeId -> | ||
| traverseMovedNode(tree.getNode(nodeId), target, dx, dy) | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
음..! 이동될 노드만 재정렬 하고 싶으셔서 따로 arrangeNode 함수에 인자를 주신 것 같아 보여요! 좋은데요??
그런데 여기서 traverseMovedNode 함수가 재귀함수고, 그럼 매번 자식 노드를 만날때마다 arrangeNode를 해주는 것 같은데 제가 기억하기로는 arrangeNode가 인자로 받은 root 기준으로 자식들을 다 끝까지 재귀로 돌았던 것 같아요. 그래서 제가 맞게 기억하고 있다면rightLayoutManager.arrangeNode(tree, target as RectangleNode) 이 부분은 traverseModeNode 안쪽보다는 moveNode 함수에서 호출해주는게 더 좋을것 같은데, 혹시 어떻게 생각하세요?
|
그리고 여기서 JetpackCompose 부분 말고도 ktlint가 걸린 부분이 있어서 고쳐지면 좋을 것 같다는 생각이 들지만.. 이대로라면 ktlint를 아예 빼버릴 수도 있어서 고쳐달라고 말씀드리기는 고민되네요🤔 회의하면서 얘기해보면 좋을 것 같아요! |
이부분 제가 대신 답변 드리자면 |
|
고생하셨습니다!! 자소서 쓰느라 바쁘셨을 텐데도 열정 넘치신 모습에 저도 자극되네요 오늘도 배워갑니다ㅎㅎ |
관련 이슈
작업한 내용