Skip to content

Conversation

@minimo-9
Copy link
Collaborator

📌 변경 사항 개요

Footer 컴포넌트 구현 및 라우터 적용 완료

📝 상세 내용

디자인만 구현 했습니다 부가 기능은 추후 시간이 된다면 sns 정도만 넣을 것 같습니다.
각 페이지마다 나오게 적용 했습니다. 피그마상 없는 부분은 제외시켰습니다.
데스크탑에서는 가운데 고정 사이즈로해서 안 넓어지게 했습니다.

🔗 관련 이슈

🖼️ 스크린샷(선택사항)

image
image

💡 참고 사항

@minimo-9 minimo-9 requested review from a user, Moon-ju-young and Yun-Jinwoo June 20, 2025 12:10
@minimo-9 minimo-9 self-assigned this Jun 20, 2025
@minimo-9 minimo-9 added the ✨ 기능 추가/구현 새로운 기능을 추가하거나 구현했어요! label Jun 20, 2025
@minimo-9 minimo-9 linked an issue Jun 20, 2025 that may be closed by this pull request
1 task
Copy link
Collaborator

@Yun-Jinwoo Yun-Jinwoo 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.

💬 footer는 항상 맨 아래에 위치해야 하므로 sticky나 fixed로 최하단에 자체적으로 위치시키는 게 좋을 것 같습니다!

Copy link
Collaborator

@Yun-Jinwoo Yun-Jinwoo Jun 20, 2025

Choose a reason for hiding this comment

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

충분히 가능한 생각이긴 하나 대부분의 웹사이트들이 footer를 sticky나 fixed로 하단에 고정하진 않는걸로 알고 있습니다.
항상 보여야하는 부분이 있으면 모르겠지만(ex. 음악 플레이어), 지금 상태에서 해당 옵션을 적용하면 괜히 화면을 가리는 느낌이 들것 같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

흠.. 한 번 생각을 해봐야겠군요 지금 안에 요소가 없어서 맨 아래 안 나오는 느낌이라..

Comment on lines +7 to +8
<div className="h-126 w-full bg-gray-10 px-20 pt-32 pb-16 font-regular text-gray-50 md:h-100 md:px-32 md:py-37 lg:flex lg:justify-center lg:px-238">
<div className="grid h-78 grid-cols-[1fr_auto] grid-rows-1 gap-x-59 gap-y-40 md:flex md:h-26 md:items-center md:justify-between lg:w-964">
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<div className="h-126 w-full bg-gray-10 px-20 pt-32 pb-16 font-regular text-gray-50 md:h-100 md:px-32 md:py-37 lg:flex lg:justify-center lg:px-238">
<div className="grid h-78 grid-cols-[1fr_auto] grid-rows-1 gap-x-59 gap-y-40 md:flex md:h-26 md:items-center md:justify-between lg:w-964">
<div className="h-126 w-full bg-gray-10 px-20 pt-32 pb-16 font-regular text-gray-50 md:h-100 md:px-32 md:py-37 lg:px-238">
<div className="grid h-78 grid-cols-[1fr_auto] grid-rows-1 gap-x-59 gap-y-40 md:flex md:h-26 md:items-center md:justify-between">

💬 이렇게 두어도 똑같은 스타일로 보이지 않나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

고정을 안하면 넓은 화면에서 사이가 벌어집니다! 그래서 넓어지게 할까 아님 고정으로 할까 고민을 했는데 상단 헤드가 고정이라 동일하게 맞췄습니다!

@minimo-9 minimo-9 merged commit 6238b76 into develop Jun 20, 2025
2 checks passed
@minimo-9 minimo-9 deleted the feat/120-footer branch June 20, 2025 13:51
@Moon-ju-young Moon-ju-young linked an issue Jun 21, 2025 that may be closed by this pull request
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ 기능 추가/구현 새로운 기능을 추가하거나 구현했어요!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] footer 컴포넌트 구현 [feat] Footer 컴포넌트 구현

4 participants