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

[FE] 목록 조회 totalPages 오류 수정 및 EmptyMemberRecord 개선 #629

Merged

Conversation

nlom0218
Copy link
Collaborator

@nlom0218 nlom0218 commented Oct 6, 2023

관련 이슈

closed #628

구현 기능 및 변경 사항

  • 기록 목록 조회 전체 페이지 버그 수정: totalPages은 1부터 시작합니다. 이를 생각하지 못하고 로직을 작성하여 수정합니다.
  • 기록이 없는 경우 Pagination 버튼 제거
  • 기록이 없는 경우 보여지는 EmptyMemberRecord 개선
    • 문구를 "완료한 스터디가 없어요. 스터디를 시작해 보세요."로 수정 -> 괜찮은 문구 있으면 추천해주세요.
    • 기존 스터디 개설하기, 스터디 참여하기 버튼을 스터디 시작하기 버튼으로 대체함. 버튼을 통해 /mode로 이동
    • 반응형 추가

스크린샷(선택)

스크린샷 2023-10-06 오후 8 49 35 스크린샷 2023-10-06 오후 8 49 45

위치 및 반응형 추가
스터디 시작하기 버튼으로 수정
@nlom0218 nlom0218 requested review from woo-jk and yeopto October 6, 2023 11:50
@nlom0218 nlom0218 self-assigned this Oct 6, 2023
@nlom0218 nlom0218 linked an issue Oct 6, 2023 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@yeopto yeopto left a comment

Choose a reason for hiding this comment

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

GOOD👍🏻

Copy link
Collaborator

@woo-jk woo-jk left a comment

Choose a reason for hiding this comment

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

스크린샷 2023-10-11 오후 9 34 03

굳굳 딱히 코드적으로 리뷰할건 없었습니다! 근데 위 스크린샷 보면 조회 버튼이 날짜 선택보다 위에있는데, 순서를 바꾸는게 더 자연스럽지 않나요!?

@nlom0218
Copy link
Collaborator Author

nlom0218 commented Oct 11, 2023

조회버튼을 달력 위에다 위치한 이유는 달력이 아래에 보이게 된다면 조회 버튼을 가리게 되기 때문입니다!

현재 날짜 조회는 조회 버튼을 눌러야 가능합니다. 때문에 조회 버튼이 아래에 있게 된다면 달력을 닫기 위해 배경을 클릭하고 조회 버튼을 클릭해야 하는데, 이러한 점이 번거로울 것 같아서 위에다 위치시켰습니다!

저도 어색하긴 해서.. 조회를 달력 내에도 만들어볼까? 했는데, 일단은 추가 기능 없이 스타일로만 해결이 가능해서 그대로 두었어요 😁

@nlom0218 nlom0218 merged commit 7c43187 into develop Oct 11, 2023
2 checks passed
@nlom0218 nlom0218 deleted the fe/refactor/628-fe-totalpages-emptymemberrecord-improvement branch October 11, 2023 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] 목록 조회 totalPages 오류 수정 및 EmptyMemberRecord 개선
3 participants