-
Notifications
You must be signed in to change notification settings - Fork 7
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
ChatPage, MainPage, PleasePage페이지를 ListLayoutPage로 공통화 #593
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.
잘 확인했습니다 고생했어요
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.
지금 '해주세요' 탭에 '안내면진다'가 들어갈 예정이에요.
안내면진다 목록 페이지에서 ListLayout 컴포넌트를 만들어서 개발했어요.
해주세요 목록은 메인 탭의 하위로 들어갈 것 같아요(팀 회의 필요).
먼저 안내면진다 기능을 머지한 뒤에, 그 부분을 이 브랜치에서 다시 반영한 다음 approve로 바꾸도록 하겠습니다..!
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.
수정 완료했습니다!
@치코
PR의 목적이 무엇인가요?
Layout에서 ChatPage, MainPage, PleasePage가 중복되는 코드가 발생한다. 이렇게 때문에 각 페이지마다 css가 달라서 페이지마다 layout이 달라지는 문제가 발생한다. 이를 해결하기 위해 공통 ListLayoutPage라는 공통 layout으로 분리한다.
이슈 ID는 무엇인가요?
설명
변경 사항이 많습니다. 차근차근 한번 살펴보아주세요!
ChatPage, MainPage, PleasePage 현재 layout이 다른 문제가 발생하고 있다.
이는 공통된 layout으로 사용할 수 있음에도 이를 재사용하지 않고 각각 css 적용하고 있어 통일되지 않고 있습니다.
이를 해결하기 위해 ChatPage, MainPage, PleasePage를 ListLayoutPage라는 공통 layout으로 분리하였습니다.
수정사항은 다음과 같습니다.
현재 MainPage
수정된 MainPage
현재 ChatPage
수정된ChatPage
현재 PleasePage
수정된 PleasePage
질문 혹은 공유 사항 (Optional)