Skip to content

feat : Web-15 신청 부원 조회 페이지#4

Open
chlgusdn0203 wants to merge 12 commits into
developfrom
web-15
Open

feat : Web-15 신청 부원 조회 페이지#4
chlgusdn0203 wants to merge 12 commits into
developfrom
web-15

Conversation

@chlgusdn0203

@chlgusdn0203 chlgusdn0203 commented Jun 1, 2026

Copy link
Copy Markdown

📌 작업 내용

신청 부원 조회 페이지 구현
신청자 목록 UI 구성
테이블 레이아웃 및 기본 스타일 적용
CSS Modules를 사용하여 페이지 스타일 분리

🔗 관련 이슈

📸 스크린샷 / 화면 녹화

image image

🧪 테스트 방법

1.http://localhost:5173/admin 로 접근
2.
3.

✅ 체크리스트

  • npm run lint 통과를 확인했습니다
  • npm run format 을 적용했습니다
  • 디버깅용 console.log 를 제거했습니다
  • 사용하지 않는 import / 변수를 정리했습니다
  • 컨벤션 (README.md) 을 준수했습니다
  • base 브랜치가 develop 인지 확인했습니다

💬 리뷰어에게 한 마디

@chlgusdn0203 chlgusdn0203 requested a review from takjinwu June 1, 2026 10:21
@takjinwu

takjinwu commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

/gemini review

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request integrates Tailwind CSS into the project, updates global styles and fonts, and implements a detailed applicant management view on the Admin page alongside a redesigned Header component. The review feedback highlights two key areas for improvement: adding a visible label to the empty admin navigation link in the Header, and enhancing the Admin page modal UX by allowing users to close it when clicking on the backdrop.

Comment thread src/components/layout/Header.jsx Outdated
Comment on lines +54 to +61
<NavLink
to={ROUTES.ADMIN_APPLICANTS}
className={({ isActive }) =>
isActive ? `${styles.navLink} ${styles.active}` : styles.navLink
}
>

</NavLink>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

관리자 확인용 임시 메뉴인 NavLink 내부에 텍스트나 자식 요소가 누락되어 있어 화면에 아무것도 표시되지 않습니다. 사용자가 클릭하여 이동할 수 있도록 링크 텍스트(예: <span>지원자 관리</span>)를 추가해 주세요.

Suggested change
<NavLink
to={ROUTES.ADMIN_APPLICANTS}
className={({ isActive }) =>
isActive ? `${styles.navLink} ${styles.active}` : styles.navLink
}
>
</NavLink>
<NavLink
to={ROUTES.ADMIN_APPLICANTS}
className={({ isActive }) =>
isActive ? `${styles.navLink} ${styles.active}` : styles.navLink
}
>
<span>지원자 관리</span>
</NavLink>

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

NavLink삭제했습니다

Comment thread src/pages/admin/AdminPage.jsx Outdated
Comment on lines +167 to +173
<div className={styles.modalBackdrop} role="presentation">
<section
className={styles.modal}
role="dialog"
aria-modal="true"
aria-label="신청 부원 상세 정보"
>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

모달 백드롭(배경) 영역을 클릭했을 때 상세 모달창이 닫히도록 설정하면 사용자 경험(UX)이 향상됩니다. 백드롭에 onClick={handleCloseDetail}을 추가하고, 모달 내부 콘텐츠 영역 클릭 시 이벤트가 상위로 전파되어 모달이 의도치 않게 닫히는 것을 방지하기 위해 onClick={(e) => e.stopPropagation()}을 추가하는 것을 권장합니다.

        <div className={styles.modalBackdrop} onClick={handleCloseDetail} role="presentation">
          <section
            className={styles.modal}
            role="dialog"
            aria-modal="true"
            aria-label="신청 부원 상세 정보"
            onClick={(e) => e.stopPropagation()}
          >

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

handleOpenDetail, handleCloseDetail 핸들러 분리 (컨벤션: handle + 동작)
백드롭 클릭 시 모달 닫힘 (onClick={handleCloseDetail})
모달 내부 클릭 시 이벤트 버블링 차단 (stopPropagation)
X 버튼도 인라인 익명 함수 → handleCloseDetail로 교체
수정완료했습니다

@chlgusdn0203 chlgusdn0203 changed the title Web 15 feat : Web-15 신청 부원 조회 페이지 Jun 8, 2026
@chlgusdn0203 chlgusdn0203 changed the base branch from main to develop June 8, 2026 08:37
@@ -1,17 +1,69 @@
import { Link } from 'react-router-dom';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image Image

Image 디자인과 다른 부분들 수정해주세요

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

추가로 픽셀 안 맞는 부분들 24px로 수정해주세요.

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image 신청 부원 조회도 figma 상에서 40px입니다. 현재는 42px에요

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

전부 수정했습니다

Comment thread src/pages/admin/AdminPage.module.css Outdated
@@ -0,0 +1,259 @@
.page {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CSS modules 사용하는것이 아닌 tailwind css만 사용해주세요. 파일 삭제 부탁드립니다

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

수정완료했습니다

@chlgusdn0203

Copy link
Copy Markdown
Author

AdminPage 파일의 내용을 RecruitManagePage 파일로 옮겼고
테스트방법은
http://localhost:5173/admin/Memberpage
로 수정했습니다

@takjinwu

Copy link
Copy Markdown
Contributor
image image NO 밑의 열 안맞는것 디자인과 다릅니다. 수정해주세요

@takjinwu

Copy link
Copy Markdown
Contributor
image 이부분 FF6B00에서 70퍼센트의 투명도를 주어야 합니다. figma 디자인 다시 확인해주세요

@takjinwu

Copy link
Copy Markdown
Contributor
image 이 부분 지원동기,사용해봤거나 들어본 언어 및 라이브러리, 동아리에서 해 보고 싶은 것,마지막으로 하고 싶은 말입니다. 화면설계서 참고해서 수정해주세요.

Comment thread src/pages/admin/RecruitManagePage.jsx Outdated

return (
<section className="min-h-[calc(100vh-112px)] bg-brand-soft px-[5.75rem] py-7 max-lg:px-6">
<div className="mx-auto min-h-[842px] max-w-[1254px] rounded-card bg-white px-[60px] pt-[52px] pb-[28px] shadow-[0_4px_4px_rgba(0,0,0,0.25)]">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

shadow값 하드코딩 금지 tailwind.config.js에서 import해서 사용해주세요.

Comment thread src/pages/admin/RecruitManagePage.jsx Outdated

{selectedApplicant && (
<div
className="fixed inset-x-0 bottom-0 top-[112px] flex items-center justify-center bg-[rgba(238,238,238,0.78)] p-8"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

rgb 토큰 이미 EEEEEE(line)으로 tailwind.config.js에 정의되어 있습니다. 수정해 주세요.

Comment thread src/pages/admin/RecruitManagePage.jsx Outdated
<nav className="mt-8 flex items-center justify-center gap-9">
<button type="button" aria-label="이전 페이지">
<svg width="24" height="24" viewBox="0 0 24 24" fill="none" aria-hidden="true">
<path d="M15 18l-6-6 6-6" stroke="#1A1A1A" strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" />

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1A1A1A 토큰 이미 ink로 정의되어있습니다 import해서 사용해주세요

Comment thread src/pages/admin/RecruitManagePage.jsx Outdated
</p>
</div>

<div className="overflow-hidden rounded-[50px] border border-tag max-lg:overflow-x-auto">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

rounded-50은 tailwind.config.js에 이미 borderRadius.card 50px 토큰으로 정의되어 있습니다. 수정해 주세요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants