코드 diff 보는 방식 변화?

[Godsenal]

#4 혜진누님이 주셨던 피드백 바탕으로 수정할지 생각해봅시다.

현재 방식은
댓글로 수정 요청 -> 반영시 댓글에 diff 표시

대신, diff가 실제로 필요한 것은 질문자의 코드니까
댓글로 수정 요청 -> 반영 -> 질문자의 원코드와 수정된 코드(마지막 수정 결과만) diff 표시

어떻게 생각하시는지들~~

[geusan]

질문자의 코드는 계속 업데이트 되고 있는데,
변경이 되어야 하는 목적이 있어?

[Godsenal]

댓글이 어떻게 반영되었는지 보다 질문자의 코드가 어떻게 바뀌었는지를 보여주는게 좋을것같다는 생각
질문자의 원코드 | 바뀐 코드
이런식으로 보여주는거

[jawsgeun]

음 현재 방식이 지금 보니 조금 이상하기도 한거같아!
두번째 방식으로 마지막 수정 결과만 보여줘도 괜찮을거 같은데?
그래야 다른 사람들이 와서 마지막 수정 결과를 보고 댓글을 달지 않을까?

[jawsgeun]

질문자의 코드는 계속 업데이트 되고 있는데,
변경이 되어야 하는 목적이 있어?

이 질문 이해가 안돼! 설명 좀!

[geusan]

지금은 기존 코드에서 리뷰가 쌓이면서 변경된 코드 상태를 유지하고 있는 거잖아.
바뀐 방식으로 보면 바뀐 부분을 알 수 있으니까 좋겠다는 생각이 들지만,
수정해야 할 부분이 한눈에 들어오지 않을 거 같다는 생각이 들어서

diff가 들어간 상태로 바꾸는게 리뷰어에게는 도움이 될지 궁금해서

개인적인 생각으로는 "얼마나 많이 바뀌었는지 확인"이 목적이 아니라 "지금 당장 고칠 것이 무엇인지 찾을 수 있음"이 목적인 화면이라고 생각해서

코드리뷰 화면에 대한 목적을 명확히한 후에 UI를 수정하면 좋을 거같아

대안으로 tab으로 구분해서 [원본보기][diff보기][최종보기] 이렇게 3단계를 주는 것도 방법 일 거 같아

[Godsenal]

3단계좋다~~!
디폴트는 최종보기로 해서 원하면 원본이랑 Diff 볼수있게