[#11] 제보사이트 - 코드 리뷰 반영 by chaerin-dev · Pull Request #12 · JCTG-JavaScript-Coding-Test-Group/SolutionBankWebsite
| </DataList> | ||
| </StepByStepInputItem> | ||
|
|
||
| {isDetailContentUnvisible ? null : ( |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
!isDetailContentUnvisible && (<></>) 와 논리곱 연산자를 써도 좋을 것 같아요! A && B 는 A가 참이여야 B를 수행하고, A가 거짓이면 아무것도 실행하지 않는 것을 뜻합니다.
| @@ -0,0 +1,27 @@ | |||
| // Tabs에서 공통적으로 사용되는 부분을 위해 폴더구조를 이렇게 만들었는데, 뭔가 어색하네요.. | |||
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분은 저도 공감해요! 아마 아래있는 style 들은 저희가 report 들에 공통적으로 들어갈 component를 만든다면 그 안에 들어갈 내용이 될 거라고 생각합니다 ㅎㅎ
| </DataList> | ||
| </StepByStepInputItem> | ||
|
|
||
| {isDetailContentUnvisible ? null : ( |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A && ()
!isDetailContentVisible && ()
앞의 조건이 true인 경우에만 뒤에 것을 시행합니다.
병합 연산자??
| @@ -7,17 +8,17 @@ import SolutionReport from "./Tabs/SolutionReport"; | |||
| export default function TabNavigation() { | |||
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아래와 같은 폴더구조를 생각해보셔도 좋을 것 같아요!!
router.js
routes.js
App.js
