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

refactor: 레이아웃 수정 #96

Merged
merged 12 commits into from
Apr 20, 2024
Merged

refactor: 레이아웃 수정 #96

merged 12 commits into from
Apr 20, 2024

Conversation

Leejin-Yang
Copy link
Contributor

@Leejin-Yang Leejin-Yang commented Apr 19, 2024

Issue

✨ 구현한 기능

  • 기존 minimal 레이아웃 삭제
  • 인디케이터 영역 추가
  • max-width 400 -> 440

📢 논의하고 싶은 내용

  • 미니멀 레이아웃 삭제했습니다. 라우터 정리도 했는데 확인 부탁드려요
  • 기존에 작성한 페이지는 아직 수정 안 했는데 바로 하면 될까요? 현재 작업에서 더 건드리는게 없는지

🎸 기타

  • 특이 사항이 있으면 작성합니다.

⏰ 일정

  • 추정 시간 : 2시간
  • 걸린 시간 : 30분

Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-qnfgiczenm.chromatic.com/

Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-umlxhdaxbc.chromatic.com/

Copy link
Contributor

@hae-on hae-on left a comment

Choose a reason for hiding this comment

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

기존에 작성한 페이지는 아직 수정 안 했는데 바로 하면 될까요? 현재 작업에서 더 건드리는게 없는지

이 말이 router에서 기존에 있던 페이지도 layout 수정하신다는 말씀일까용??

Comment on lines 26 to 27
<span className={isActive ? menuName['active'] : menuName['default']}>{name}</span>
</>
Copy link
Contributor

Choose a reason for hiding this comment

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

요기 Text 컴포넌트로 대체 가능한가요???

Copy link
Member

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

와 레이아웃 드디어! 고생하셨슴다

@Leejin-Yang Leejin-Yang merged commit 525adb7 into feat/v2 Apr 20, 2024
2 of 3 checks passed
@Leejin-Yang Leejin-Yang deleted the feat/issue-90 branch April 20, 2024 08:26
Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-qmcbqydtdc.chromatic.com/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

레이아웃 수정
3 participants