Skip to content

feat: 전반적인 홈 스타일 개선(테마 스타일링, 사이드바 추가)#2

Merged
JunilHwang merged 6 commits intohanghae-plus:mainfrom
YeongseoYoon-hanghae:feat/home-design
Aug 2, 2025
Merged

feat: 전반적인 홈 스타일 개선(테마 스타일링, 사이드바 추가)#2
JunilHwang merged 6 commits intohanghae-plus:mainfrom
YeongseoYoon-hanghae:feat/home-design

Conversation

@YeongseoYoon
Copy link
Collaborator

@YeongseoYoon YeongseoYoon commented Aug 2, 2025

🚀 Pull Request

📝 변경사항 요약

전반적으로 홈 스타일을 개선했습니다. 사실 홈을 만들고 수강생 목록으로 현재 홈에 해당하는 부분을 이동시키려고 했는데, 그렇게하면 pr이 너무 커질거같아서 안했습니다...
image
이게 메인페이지라고 하셔서 이걸 하려면...일단 메인테이너의 허락을 받아야...

🎯 변경 이유 및 배경

  • 문제점: 못생김
  • 해결방안: 지금 제가 해결함
  • 기대효과: 이뻐짐

🔧 주요 변경 내용

  • shadcn을 통해 sidebar컴포넌트 관련 컴포넌트를 생성하였습니다.
  • BaseLayout에 Sidebar가 보이도록 추가구현하였습니다.
  • 전반적인 색상 테마를 변경하였습니다. (항해 추구미)

📸 스크린샷 (UI 변경시)

Before

image

After

image

🔗 관련 이슈

📚 추가 정보

컴포넌트부는 shadcn이 생성해준 부분입니다. 누가 일단 하고나서 용서를 구하는게 빠르다고 하셔서 올립니다. 우하하

image 증거도 잇음

@YeongseoYoon
Copy link
Collaborator Author

YeongseoYoon commented Aug 2, 2025

아 리뷰어 지정을 못하네 @JunilHwang

아 과제 재활용 레포였구나...어쩐지...ci가 외 도나했네

@YeongseoYoon-hanghae YeongseoYoon-hanghae force-pushed the feat/home-design branch 2 times, most recently from d8606f3 to a7b61ce Compare August 2, 2025 17:48
Copy link
Contributor

@JunilHwang JunilHwang 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

Choose a reason for hiding this comment

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

이거 안 쓰는데 왜있어요 ㄷㄷ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

저거 shadcn이 만들어줌...
그 잘 보시면 있어요 사이드바 컴포넌트에서 쓰더라고여 왜 css로 안했는지 몰겠음

Copy link
Contributor

Choose a reason for hiding this comment

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

파일이 길어서 PR에 안보였네 ㅋㅋ;

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

눈을 크게 뜨셨어야죠

@JunilHwang JunilHwang merged commit b9e6a3c into hanghae-plus:main Aug 2, 2025
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants