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

[Feature] 나의 스터디 페이지 UI 구축 #35

Merged
merged 50 commits into from
Aug 20, 2024

Conversation

ghdtjgus76
Copy link
Collaborator

🎉 변경 사항

나의 스터디 페이지 UI 부분 완성했습니다.
아직 api 연결 전이라 지금 올라온 api 기반으로 임시 목데이터 작성해뒀습니다. (올라오지 않은 api는 제가 임시로 만들어놨습니다.)

현재 next/image Image 컴포넌트로 모든 svg 파일들 넣어놨는데, 개발 완료한 후 한 번에 반영할 예정입니다..

🚩 관련 이슈

🙏 여기는 꼭 봐주세요!

ghdtjgus76 and others added 30 commits August 15, 2024 19:57
@ghdtjgus76 ghdtjgus76 linked an issue Aug 19, 2024 that may be closed by this pull request
1 task
Copy link

Copy link

Copy link

@eugene028
Copy link
Collaborator

스크린샷 2024-08-20 오전 1 10 17

캐러셀 부분을 오른쪽으로 이동시키면 다시 왼쪽으로 이동시킬 수 있는 버튼이 뜨지 않는 것 같아요!!

@ghdtjgus76
Copy link
Collaborator Author

스크린샷 2024-08-20 오전 1 10 17

캐러셀 부분을 오른쪽으로 이동시키면 다시 왼쪽으로 이동시킬 수 있는 버튼이 뜨지 않는 것 같아요!!

피그마 상에서 왼쪽 버튼이 없어서 일단은 그대로 구현해놨어요!
가로 스크롤할 수 있게 설정해뒀는데 어색한가요..?

Comment on lines +6 to +16
const handleScroll = (distance: number) => {
if (!containerRef.current) {
return;
}

containerRef.current.scrollBy({
left: distance,
behavior: "smooth",
});
};

Copy link
Collaborator

Choose a reason for hiding this comment

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

P3;
왼쪽, 오른쪽으로 handleScroll을 다른 핸들러 함수로 구현하면 더 좋을 것 같아요!
containerRef의 scrollBy객체에 distance 길이만큼 마이너스 값을 주면 될 것 같습니당.
그리고 스크롤의 시작점 이외에는 왼쪽 캐러셀 버튼이 활성화되도록 해도 좋을 것 같아요!

apps/client/types/entities/myStudy.ts Outdated Show resolved Hide resolved
Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

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

수고하셨어요~!!

과제 관련 dto 가 assignment 라는 명칭으로 되어있어서 homeworkassignment 가 섞여서 한 단어랑 통일되면 좋을 듯 한데 어떻게 생각하시나요?! 괜찮으시다면 제 PR 쪽에서도 고쳐볼게요 ㅎ,,ㅎ

apps/client/constants/homeworkSubmissionStatusMap.ts Outdated Show resolved Hide resolved
apps/client/constants/attendanceStatusMap.ts Outdated Show resolved Hide resolved
@eugene028
Copy link
Collaborator

스크린샷 2024-08-20 오전 1 10 17
캐러셀 부분을 오른쪽으로 이동시키면 다시 왼쪽으로 이동시킬 수 있는 버튼이 뜨지 않는 것 같아요!!

피그마 상에서 왼쪽 버튼이 없어서 일단은 그대로 구현해놨어요! 가로 스크롤할 수 있게 설정해뒀는데 어색한가요..?

앗 가로 스크롤이 가능하군여ㅎㅎ... 스크롤 안해봐서 확인을 못했던 것 같아요! 🥹 피그마 UI에 오른쪽 화살표만 만들어 두셨다면 현재 개발하신대로 가는게 맞을거같아요!!

@ghdtjgus76
Copy link
Collaborator Author

수고하셨어요~!!

과제 관련 dto 가 assignment 라는 명칭으로 되어있어서 homeworkassignment 가 섞여서 한 단어랑 통일되면 좋을 듯 한데 어떻게 생각하시나요?! 괜찮으시다면 제 PR 쪽에서도 고쳐볼게요 ㅎ,,ㅎ

스터디 페이지는 assignment로 반영해뒀습니다!

Copy link

Copy link

Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

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

좋습니다 수고 많으셨어요~!!

@ghdtjgus76 ghdtjgus76 merged commit 7afab22 into dev Aug 20, 2024
3 of 4 checks passed
@ghdtjgus76 ghdtjgus76 deleted the feature/my-study-layout branch August 20, 2024 10:00
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.

[Feature] 나의 스터디 페이지 레이아웃 구현
3 participants