Skip to content
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

Merged
merged 7 commits into from
Oct 15, 2024

Conversation

jaeml06
Copy link
Contributor

@jaeml06 jaeml06 commented Sep 30, 2024

PR의 목적이 무엇인가요?

Layout에서 ChatPage, MainPage, PleasePage가 중복되는 코드가 발생한다. 이렇게 때문에 각 페이지마다 css가 달라서 페이지마다 layout이 달라지는 문제가 발생한다. 이를 해결하기 위해 공통 ListLayoutPage라는 공통 layout으로 분리한다.

이슈 ID는 무엇인가요?

설명

변경 사항이 많습니다. 차근차근 한번 살펴보아주세요!
ChatPage, MainPage, PleasePage 현재 layout이 다른 문제가 발생하고 있다.
이는 공통된 layout으로 사용할 수 있음에도 이를 재사용하지 않고 각각 css 적용하고 있어 통일되지 않고 있습니다.
이를 해결하기 위해 ChatPage, MainPage, PleasePage를 ListLayoutPage라는 공통 layout으로 분리하였습니다.
수정사항은 다음과 같습니다.

  • 불필요한 css 삭제
  • ChatPage, MainPage, PleasePage를 ListLayoutPage라는 공통 layout으로 분리
  • 새로고침 svg 변경
  • mainPage 헤더 네비게이션 UI 수정
    현재 MainPage
image

수정된 MainPage
image


현재 ChatPage
image

수정된ChatPage
image


현재 PleasePage

image

수정된 PleasePage
image

질문 혹은 공유 사항 (Optional)

@jaeml06 jaeml06 added FE 프론트엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정) labels Sep 30, 2024
@jaeml06 jaeml06 self-assigned this Sep 30, 2024
Copy link
Contributor

@ss0526100 ss0526100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

잘 확인했습니다 고생했어요

Copy link
Contributor

@cys4585 cys4585 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

지금 '해주세요' 탭에 '안내면진다'가 들어갈 예정이에요.
안내면진다 목록 페이지에서 ListLayout 컴포넌트를 만들어서 개발했어요.
해주세요 목록은 메인 탭의 하위로 들어갈 것 같아요(팀 회의 필요).

먼저 안내면진다 기능을 머지한 뒤에, 그 부분을 이 브랜치에서 다시 반영한 다음 approve로 바꾸도록 하겠습니다..!

Copy link
Contributor

@cys4585 cys4585 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수정 완료했습니다!
@치코

@cys4585 cys4585 merged commit 2f7a331 into develop-frontend Oct 15, 2024
1 check passed
@ss0526100 ss0526100 deleted the refactor/#590 branch October 17, 2024 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정)
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants