Repository navigation
[admin] 지원자 목록 페이지네이션에 첫·끝 페이지·말줄임표·전체 인원 표시 - #561
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
s2yeons
approved these changes
Oct 20, 2026
![pr-agent-demo[bot]](https://avatars.githubusercontent.com/in/3949189?s=60&v=4){kind=small}
There was a problem hiding this comment.
-
Overview: 어드민 지원자 목록의 페이지네이션을 개선하는 PR입니다. 첫/끝 페이지·말줄임표 표시, “총 N명” 표시, 페이지 이동 UX를 보완하고 기존 내장 로직을 별도 컴포넌트로 분리합니다.
-
Intent: 기존에 현재 페이지 근처 번호만 보여 전체 페이지 수와 인원이 불분명했던 문제를 해결하고, 접근성·키보드 조작·새 탭 열기·스크롤 초기화 등 사용성을 개선하려는 의도로 보입니다.
-
Risk: 전반적으로 낮음~보통입니다. 페이지 번호 계산의 경계값, 말줄임표 로직, URL 쿼리 반영, 클릭/키보드/새 탭 동작, 테이블 스크롤 초기화, 기존 필터·정렬 관련 변경과의 병합 충돌 여부를 중점적으로 확인하면 좋습니다.
0 inline comment(s)
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
개요 💡
어드민 지원자 목록 하단 페이지네이션이 전체 페이지 범위와 인원을 보여주도록 개선하였습니다. 지금까지는 현재 페이지 근처 번호 3개와 화살표만 보여(
‹ 1 2 3 ›) 페이지가 3개뿐인 것처럼 보이고, 전체 인원이나 마지막 페이지를 알 수 없었습니다.작업내용 ⌨️
페이지 번호 계산
getPaginationItems(currentPage, totalPages)를 추가하였습니다. 첫·끝 페이지는 항상 표시하고 현재 페이지 앞뒤로 2개씩 표시하며, 전체 9페이지 이하면 말줄임 없이 모두 표시합니다.1 … 3) 그 번호를 대신 표시하여, 10페이지 이상에서는 항상 9칸으로 유지되어 페이지를 넘길 때 버튼 위치가 흔들리지 않습니다.tsx --test단위 테스트 7개를 추가하였습니다. 10~30페이지의 모든 현재 페이지에 대해 항목 수, 현재 페이지 포함 여부, 말줄임표가 2개 이상을 가리는지 검사합니다.ApplicantPagination컴포넌트MainPage의 페이지네이션을 컴포넌트로 분리하고총 N명(info.totalElements)을 표시하였습니다.‹, 마지막 페이지의›는aria-disabled와 클릭 방지 스타일로 비활성화하였습니다.href를 부여하여 키보드 Tab으로 이동하고 Enter로 선택할 수 있게 하였습니다. 일반 클릭은preventDefault후 기존처럼 URL만 교체하고, Ctrl·Cmd 클릭은 새 탭으로 열립니다. 공용PaginationLink는 수정하지 않았습니다.관련 이슈 🚨
리뷰 요청사항 👀
1 2 3 4 5 6 7 … 23,1 … 4 5 6 7 8 … 23,1 … 17 … 23, 9페이지 이하 전체 표시, 첫/끝 화살표 비활성, 키보드 이동, 스크롤 초기화, 전체 새로고침이 일어나지 않는 것을 확인하였습니다.총 N명위치는 이슈 예시대로 페이지 번호 위에 두었습니다. 디자인이 따로 있다면 맞추겠습니다.MainPage를 수정하지만, 세 PR([task] 어드민 지원자 목록 필터에 '전체' 옵션 추가 #555·[task] 어드민 지원자 목록 페이지네이션 UI 개선 (첫/끝 페이지·말줄임표·전체 인원 표시) #556·[task] 어드민 지원자 목록 컬럼 헤더 클릭 정렬 추가 #558 대응)을 함께 병합해도 충돌 없이 타입 검사·테스트가 통과하는 것을 확인하였습니다.