Skip to content

[Feat] 헤더 컴포넌트 수정#57

Merged
sispo3314 merged 5 commits intodevelopfrom
feat/#56/헤더-컴포넌트-수정
Aug 4, 2025

Hidden character warning

The head ref may contain hidden characters: "feat/#56/\ud5e4\ub354-\ucef4\ud3ec\ub10c\ud2b8-\uc218\uc815"
Merged

[Feat] 헤더 컴포넌트 수정#57
sispo3314 merged 5 commits intodevelopfrom
feat/#56/헤더-컴포넌트-수정

Conversation

@sispo3314
Copy link
Copy Markdown
Collaborator

@sispo3314 sispo3314 commented Aug 4, 2025

🔍 관련된 이슈

📝 작업 내용

  • 헤더컴포넌트에서 className을 prop으로 받을 수 있도록 수정했습니다.
  • figma에서는 헤더 높이가 44px인데 y 패딩 때문인지 64px로 보여서 수정했습니다! (혹시 의도하고 이렇게 해두신 거라면 말씀해주세요!! 다시 수정하겠습니다)
  • 리뷰 페이지에서 헤더가 이미지 영역을 지나면 bg-gray-900으로 보이도록 수정했습니다.
  • 이외 헤더가 있는 페이지에서 모두 헤더를 gray-900으로 지정해두었는데 적용이 안되었거나 잘못된 게 있다면 수정해주세용...

📸 스크린샷

2025-08-04.233457.mp4
image

🚨 이슈

📣 리뷰 요구사항

✅ 체크리스트

  • 코드가 정상적으로 컴파일되나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?
  • 리뷰어를 지정했나요?

@sispo3314 sispo3314 self-assigned this Aug 4, 2025
@sispo3314 sispo3314 added the 🎨Style 스타일 추가 및 수정 label Aug 4, 2025
@sispo3314 sispo3314 linked an issue Aug 4, 2025 that may be closed by this pull request
3 tasks
Copy link
Copy Markdown
Member

@dalzzy dalzzy left a comment

Choose a reason for hiding this comment

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

굿굿 원래 이 부분 제가 할려고했는데 ,, 감사합니다 허허
수정할 부분 없는 것 같아서 approve 했습니다~~

Comment on lines -26 to +34
<header className="fixed top-0 right-0 left-0 z-50 mx-auto flex h-[44px] w-full max-w-[430px] items-center justify-between bg-gray-900 px-5 py-8">
<header
className={cn(
'fixed top-0 right-0 left-0 z-50 mx-auto flex h-[44px] w-full max-w-[430px] items-center justify-between px-5',
className,
)}
>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

아 기본으로 설정해둔 배경색을 없애고 className으로 주도록 수정하신거군용??

Copy link
Copy Markdown
Collaborator Author

@sispo3314 sispo3314 Aug 4, 2025

Choose a reason for hiding this comment

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

넵!! 더 좋은 방법이 있었을 거 같은데 빠르게 하려다보니... ㅎㅎ

Copy link
Copy Markdown
Collaborator

@soyun-git121 soyun-git121 left a comment

Choose a reason for hiding this comment

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

저도 헤더부분이 큰거같다고 생각했는데 수정해주셔서 감사합니당

@sispo3314 sispo3314 merged commit efef06b into develop Aug 4, 2025
1 check passed
@dalzzy dalzzy deleted the feat/#56/헤더-컴포넌트-수정 branch August 8, 2025 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨Style 스타일 추가 및 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] #56 헤더 컴포넌트 수정

3 participants