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

[Feat] 소개페이지 업데이트 #410

Merged
merged 8 commits into from
Sep 6, 2024
Merged

[Feat] 소개페이지 업데이트 #410

merged 8 commits into from
Sep 6, 2024

Conversation

wuzoo
Copy link
Contributor

@wuzoo wuzoo commented Sep 5, 2024

Summary

close #409

Screenshot

2024-09-05.9.47.41.mov

Comment

  • 헤더 이미지 변경
  • 핵심가치 이미지 디자인 수정
  • 커리큘럼 디자인 수정
  • 임원진 소개 mock 데이터 변경

Copy link

height bot commented Sep 5, 2024

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Member

@eonseok-jeon eonseok-jeon 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
Member

@eonseok-jeon eonseok-jeon Sep 5, 2024

Choose a reason for hiding this comment

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

스크린샷 2024-09-05 오후 10 00 01

이미지 크기와 실제 렌더링 되는 사이즈 간격이 많이 차이가 나네요
이러면 성능 안 좋아질 수 있으니 2~3배 정도로 뽑으면 될 거 같아요 :)
(사진이 작아서 2배 정도만 해도 될 듯 합니다)

사이트가 필요하시면 여기 이용하시면 될 거 같아요

Copy link
Member

Choose a reason for hiding this comment

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

헉 그러네요 한 사진당 1MB가 다 넘어가서🫨🫨🫨🫨 제 PR에도 너무 rich 하다고 안떠요 ㅠㅠ (주용님 컴의 문제가 아니엇네요)

그리고 로컬에서 실행해도 이미지들이 안받아와져서 먼진선생님들 얼굴도 34기분들로 뜨네여 ㅠㅠ
꼭 줄여야할 것 같슴두 이미지 리사이징 하시고 다시 불러주시면 확인해보겠습니당

Copy link
Member

@lydiacho lydiacho 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
Member

Choose a reason for hiding this comment

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

헉 그러네요 한 사진당 1MB가 다 넘어가서🫨🫨🫨🫨 제 PR에도 너무 rich 하다고 안떠요 ㅠㅠ (주용님 컴의 문제가 아니엇네요)

그리고 로컬에서 실행해도 이미지들이 안받아와져서 먼진선생님들 얼굴도 34기분들로 뜨네여 ㅠㅠ
꼭 줄여야할 것 같슴두 이미지 리사이징 하시고 다시 불러주시면 확인해보겠습니당

Copy link
Member

Choose a reason for hiding this comment

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

image

요 넘버링 친구들 배경색도 바꿔줘야할 것 같아요 ! ! (지금은 누리끼리)
rgba(58, 81, 109, 1) 이겁니당

Copy link
Member

Choose a reason for hiding this comment

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

image

여기 워딩도 35기 AND SOPT로 고쳐야해요!!

Copy link
Member

@lydiacho lydiacho left a comment

Choose a reason for hiding this comment

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

라스트팡입니다 ~~

@@ -18,7 +18,7 @@ const MemberSection = ({ generation }: MemberSectionProps) => {
<SectionTop
engTitle="Executives"
korTitle={`${generation}기 임원진`}
description="200명의 활동 회원들이 열정을 외칠 수 있도록, 34기 NOW SOPT를 이끄는 임원진들이에요."
description="200명의 활동 회원들이 열정을 외칠 수 있도록, 35기 NOW SOPT를 이끄는 임원진들이에요."
Copy link
Member

Choose a reason for hiding this comment

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

ㅋ ㅋ 나우솝트를 놓아주세요. 앤드솝트입니두

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋ 죄송합니다 .. 넵

@@ -1,3 +1,5 @@
'use client';
Copy link
Member

Choose a reason for hiding this comment

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

이거 아까 빌드에러 잡느라 넣어줬던거죠?? 그럼 없애줘도 될것같아요!!

Copy link
Member

@lydiacho lydiacho left a comment

Choose a reason for hiding this comment

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

구우우우우우욷 아주 수고 많았습니다 ~~ 🚀 🚀 🚀

내일 서버 prod 배포 완료되면 CI 잘 돌아가는지 최종 확인하고 바로 머지하시죵 !

Copy link
Member

@eonseok-jeon eonseok-jeon left a comment

Choose a reason for hiding this comment

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

고생하셨어요 👍

@wuzoo wuzoo merged commit a074e35 into develop Sep 6, 2024
1 check passed
@wuzoo wuzoo deleted the feat/#409_35-about branch September 6, 2024 04:57
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] 소개페이지 업데이트
3 participants