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: 멤버 페이지 구현 #21

Merged
merged 12 commits into from
Apr 18, 2024
Merged

Feat: 멤버 페이지 구현 #21

merged 12 commits into from
Apr 18, 2024

Conversation

bunju20
Copy link
Contributor

@bunju20 bunju20 commented Apr 18, 2024

PR 타입 ( 하나 이상의 PR 타입을 선택해주세요 )


  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

개요

멤버 페이지 생성했습니다.

image image

변경 사항

대략적인 뷰는 만들어놓고 타임라인 누를때마다 다른 세대 gdsc 나올수있게 세팅해놨습니다.
코드 확인받고나면 디자인은 피그마에 완전히 맞게 수정하고, 모바일 반응형 구현하겠습니다!

코드 리뷰시 참고 사항

  • 2nd 페이지는 서현님 피드백 상 배경사진을 없앤 배너를 가져오는게 낫겠다고 말씀해주셔서 확인받고나면 컴포넌트 common에 해당 배너 올려서 페이지 수정하겠습니다.
  • src/app/member/page.tsx와 src/components/member/generation/GenerationPage.tsx를 한 페이지에서 구현하는 방법이 있을지 궁금합니다. metadata랑 'use client'를 같이 쓰질 못해서 일단 두부분으로 나눠뒀습니다.
  • 웹이 아직 서툴어서 피드백... 주시면 감사하게받겠습니다...!

@bunju20 bunju20 self-assigned this Apr 18, 2024
Copy link
Member

@seochan99 seochan99 left a comment

Choose a reason for hiding this comment

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

고생하셨어용! 자잘한거 리펙토링 내용은 다음번에 추가하면될듯!

>
<Image
src={BannerPng}
alt="BannerPng"
Copy link
Member

Choose a reason for hiding this comment

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

alt 잘쓰기!

Comment on lines +21 to +32
<div className="text-4xl mb-[1.25rem]">Lead</div>
<div className='flex'>
<div className="text-3xl mr-[0.5rem]">서희찬</div>
<div className="text-2xl mb-[1.25rem]">컴퓨터공학과</div>
</div>
<div className="flex items-center text-lg">
<div className="w-1 h-12 bg-white mr-2.5"></div>
<div>
안녕하세요! GDSC DGU Lead 서희찬입니다.
<br/>
GDSC DGU는 많은 학우분들과 함께 성장하고 싶습니다.
많은 관심 부탁드립니다! 감사합니다.
Copy link
Member

Choose a reason for hiding this comment

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

const데이터로 뺴기

Comment on lines +22 to +42
const [selected, setSelected] = useState(1);

const handleCircleClick = (index: any) => {
if (index === 0) return;
setSelected(index);
onSelectIndex(index); // Pass the selected index up to the parent component
};

const scrollContainer = useRef(null);

const getCircleColor = (index: any) => {
if (index === selected) return '#FB8C00';
if (index < selected) return '#FFCC80';
return '#9FA4A8';
};

const getTextColor = (index: any) => {
if (index === selected) return '#FFFFFF';
return '#9FA4A8';
};

Copy link
Member

Choose a reason for hiding this comment

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

약간 이런 로직은 다른 파일로 뺼수없나 한번 봐야할듯!

@seochan99
Copy link
Member

src/app/member/page.tsx와 src/components/member/generation/GenerationPage.tsx를 한 페이지에서 구현하는 방법이 있을지 궁금합니다. metadata랑 'use client'를 같이 쓰질 못해서 일단 두부분으로 나눠뒀습니다.

metadata를 포기하면 됩니다 ㅎㅎ.ㅎ.ㅎ.ㅎ.ㅎ....ㅠㅠ

Copy link
Member

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

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

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

@shimseohyun shimseohyun left a comment

Choose a reason for hiding this comment

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

맴버 넣느라 고생 많으셨습니다... ... 이거 앞으로도 하나하나 수작업해야하나요?

@bianbbc87 bianbbc87 self-requested a review April 18, 2024 15:13
@seochan99 seochan99 merged commit 9d2aecb into GDSC-DGU:dev Apr 18, 2024
1 check failed
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.

5 participants