[FE]피드백 모달 수정

[sangkyu39]

피드백 문구 수정 및 위치 조정

Summary by CodeRabbit

변경사항

• 개선사항

  • 피드백 모달 설명에 "피드백은 익명으로 저장됩니다" 문장이 추가되어 개인정보 안내가 명확해졌습니다.
  • 뷰포트 기반 높이 변수(--vh)를 적용해 창 크기 변경 시 레이아웃 안정성이 향상되었습니다.

• 스타일

  • 모바일 오버레이 및 루트 컨테이너의 높이 계산이 개선되어 화면 채움 동작이 일관됩니다.
  • 피드백 영역 상단 여백이 감소되어 레이아웃이 더 컴팩트해졌습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 42bd744b-0451-4fe3-9dce-6183cdd95edb

📥 Commits

Reviewing files that changed from the base of the PR and between eae0e84 and 35c2c18.

📒 Files selected for processing (1)

• frontend/src/App.jsx

---

Walkthrough

피드백 모달 설명에 익명 저장 안내 문구를 추가하고, 뷰포트 기반 높이 변수(--vh)를 도입해 루트 및 모바일 오버레이 높이 계산 방식을 변경했습니다. 사이드바 피드백 트리거의 상단 패딩이 제거되었고, App.jsx에 윈도우 리사이즈로 --vh를 갱신하는 useEffect가 추가되었습니다.

Changes

Cohort / File(s)	Summary
피드백 모달 텍스트 frontend/src/components/Sidebar.jsx	피드백 모달 설명(<p id="feedback-modal-description">)에 "피드백은 익명으로 저장됩니다" 문장 추가 (UI 동작/로직 변경 없음).
사이드바 스타일 frontend/src/components/Sidebar.module.css	.feedbackTriggerWrap에서 padding-top: 12px; 제거로 상단 간격 감소; 모바일 .overlay 높이 100vh → calc(100 * var(--vh))로 변경.
루트 및 뷰포트 변수 frontend/src/App.jsx, frontend/src/index.css	App.jsx에 마운트 시 및 resize 이벤트에서 --vh를 계산·설정하는 useEffect 추가(리스너 정리 포함). #root 스타일: min-height: 100vh → height: calc(100 * var(--vh)), margin: 0, padding: 0 명시.

Sequence Diagram(s)

(생성 조건 미충족 — 생략)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• [Fe] 317 버그출석관리 세션 삭제 안 열림 이슈 #318: 피드백 모달 텍스트 및 뷰포트(--vh) 관련 스타일 변경과 중복/연관성 있음.
• [FE] SISC1-198 [FEAT] 헤더 적용 및 토글 사이드바 전체 페이지 적용 #81: Sidebar 컴포넌트 및 스타일 변경과 코드 중첩 가능성 있음.

Suggested reviewers

• discipline24

시

🐰 창가에 귀 기울이며,
익명 속 말을 조심스레 남겨요.
높이의 숨결을 재어 맞추고,
틈을 줄여 더 편안히 달려요.
작은 수정에 깡충, 기쁜 춤을.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 피드백 모달 수정을 언급하지만, 실제 변경 사항은 피드백 문구 추가, CSS 레이아웃 개선(vh 단위 수정), 피드백 트리거 패딩 제거 등 여러 기능 변경을 포함하고 있습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 202603223_#324-버그fe-스크롤-버그-수정-필요

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

사이드바의 피드백 모달 안내 문구를 수정하고, 하단 피드백 트리거 영역의 레이아웃(여백)을 조정하는 프론트엔드 변경입니다.

Changes:

• 피드백 모달 설명 문구를 확장(익명 저장 안내 추가)
• 사이드바 하단 피드백 트리거 영역의 상단 padding 제거로 위치/간격 조정

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
frontend/src/components/Sidebar.jsx	피드백 모달 설명 문구 업데이트
frontend/src/components/Sidebar.module.css	피드백 트리거 영역의 상단 여백 제거로 위치 조정

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@frontend/src/App.jsx`:
- Around line 41-48: The resize listener is registered directly in the component
body causing accumulating listeners and memory leaks; move the logic into a
React useEffect hook: compute initial vh and call
document.documentElement.style.setProperty('--vh', `${vh}px`) inside the effect,
register the handler via window.addEventListener('resize', onResize) where
onResize recomputes vh and sets the CSS variable, and return a cleanup function
that calls window.removeEventListener('resize', onResize'); reference the
existing window.addEventListener, document.documentElement.style.setProperty,
and the vh calculation to implement this change.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: acddb58e-17f9-4184-9dee-8fc0ac5d5d26

📥 Commits

Reviewing files that changed from the base of the PR and between 6e73604 and eae0e84.

📒 Files selected for processing (3)

• frontend/src/App.jsx
• frontend/src/components/Sidebar.module.css
• frontend/src/index.css

🚧 Files skipped from review as they are similar to previous changes (1)

• frontend/src/components/Sidebar.module.css

[discipline24]

고생하셨습니다~