-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TripList 페이지 구현 #51
TripList 페이지 구현 #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 반영 해 보았습니다~!
확인 한번 부탁드릴게요👍
width: `calc((100vw - 196px) / 5)`, | ||
alignSelf: 'start', | ||
margin: '0', | ||
padding: '0', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전 별로 상관없긴한데 string으로 통일하는건 좀 별로려나요?🤔
'-ms-overflow-style': 'none', | ||
scrollbarWidth: 'none', | ||
'& > span': { | ||
marginRight: Theme.spacer.spacing2, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'& > span'
내부 속성들도 위 속성들 내부로 순서가 적용되는거였나요..?
'& > span'
같은건 css순서 별개로 밑에 몰아두는게 보는데 좋을 것 같은데 어떤가요?🤔
+ 스토리북 코드는 내일까지 작성하겠습니다~ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
86a6388
to
88b6a17
Compare
📄 Summary
트립 목록 화면
튜토리얼 모달 화면
🙋🏻 More
튜토리얼 모달
튜토리얼 모달에 대해, 존재하는 trip이 없을 경우 띄워주도록 우선 설정했습니다.
그렇다보니 현재 msw 모킹에 대해서 trip을 0개 반환하는 msw 핸들러와 trip들을 반환하는 핸들러로 나뉘어있습니다. 요청부분만 조금 다르게 해서 테스트할 수 있지 않을까 싶습니다.
close #36