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

[GGFE-50] Header Menu Noti 스타일 적용 #823

Merged

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Jun 13, 2023

📌 개요

  • Header Menu Noti 스타일 적용

💻 작업사항

  • Header
    프로필 사라짐, 노티는 알림 개수 보여줌 (9개 넘어가면 9+로 보여줌), 공지는 사이트링크 대신 모달로 변경

  • Menu
    42gg로고, 프로필 추가, 메뉴목록마다 항목이미지 추가

  • Noti
    알림이 없는 경우 "알림이 없습니다 + 이모지", 알림 타입마다 알림색깔이 달라짐
    기존에는 작게 알림타입 표시했었음. 경기임박, 매칭성사 등이 사라지고 색깔로 변경), 아래쪽에 물방울데코 추가됨

  • png 이미지 svg로 변경

  • ❗️noti ischecked 변경을 위한 put api 추가

  • ❗️canceledByMan 안 받는 줄 알았는데 받는 타입이라 추가

  • 사용되는 폰트 및 이미지 추가됨

✅ 변경로직

@PHJoon PHJoon self-assigned this Jun 13, 2023
Copy link
Contributor

@parksangmin1543 parksangmin1543 left a comment

Choose a reason for hiding this comment

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

메뉴바에서 사진과 이름을 눌렀을때 이동은 하지만 메뉴바가 사라지지않습니다

@PHJoon
Copy link
Contributor Author

PHJoon commented Jun 13, 2023

메뉴바에서 사진과 이름을 눌렀을때 이동은 하지만 메뉴바가 사라지지않습니다

사진과 이름 눌렀을 때 메뉴바 꺼지는 동작 추가했습니다!

@yoouyeon
Copy link
Member

수고하셨습니다!! 노티 종류 말풍선 색깔로 구분하는거 너무 좋네요👍

근데
notibar에서 IMMINENT랑 MATCHED일때에는 말풍선 배치가 의도하신대로 잘 뜨는 것 같았는데 ANNOUNCE 알림을 확인했을때에는 아래 캡쳐처럼 간격이 다르게 나오네요...!
확인 한번 해보시면 좋을 것 같습니당... 😆

Screen_Shot 2023-06-15 16 00 53 Screen_Shot 2023-06-15 16 03 45

@42sungwook
Copy link
Contributor

수정된 내용 체크했습니다 :)

@42sungwook 42sungwook merged commit 9c96f31 into GGFE-00/Feat/api-refactoring Jun 15, 2023
@PHJoon PHJoon deleted the GGFE-50-Style-Header-Modal-Menu branch June 16, 2023 03:07
@yoouyeon yoouyeon linked an issue Jun 28, 2023 that may be closed by this pull request
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.

[Bug] Menu button 클릭 범위
4 participants