Skip to content

refactor: 타이틀 타이포그라피로 변경 및 각 페이지 title 상속 제거#118

Merged
KwonDeaGeun merged 1 commit intodevelopfrom
refactor/unity-title-AEGIS-typography
Feb 8, 2026
Merged

refactor: 타이틀 타이포그라피로 변경 및 각 페이지 title 상속 제거#118
KwonDeaGeun merged 1 commit intodevelopfrom
refactor/unity-title-AEGIS-typography

Conversation

@KwonDeaGeun
Copy link
Contributor

@KwonDeaGeun KwonDeaGeun commented Feb 8, 2026

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능

    • 애플리케이션 헤더를 "AEGIS"로 통일했습니다.
    • Stack Sans Notch 폰트를 추가했습니다.
  • 스타일

    • 포맷팅 설정을 개선하고 확장했습니다.
    • CSS 파서 구성을 추가했습니다.
  • Chores

    • 코드 포맷팅 및 구조를 개선했습니다.

@KwonDeaGeun KwonDeaGeun self-assigned this Feb 8, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 8, 2026

Walkthrough

헤더 컴포넌트의 title prop을 제거하고 고정된 "AEGIS" 로고로 대체합니다. 해당 prop 사용을 여러 페이지 컴포넌트에서 제거하며, 설정 파일을 업데이트하고 새로운 폰트를 추가합니다.

Changes

Cohort / File(s) Summary
Configuration 및 Formatting
biome.json, src/index.css, src/hooks/useExternalBrowser.ts, src/pages/BrowserRedirectPage.tsx
Biome 설정을 확장된 JSON 형식으로 재작성하고 formatter, javascript, css 섹션 추가. "Stack Sans Notch" 폰트 @font-face 규칙 추가. 빈 줄 추가 및 Label 컴포넌트 JSX 포맷팅 변경.
Header 컴포넌트 개선
src/components/ui/Header.tsx, src/pages/ForbiddenPage.tsx
Header 컴포넌트에서 title prop 제거 및 "AEGIS" 하드코딩된 레이블로 변경. ForbiddenPage에서도 title prop 제거 및 Header 호출 시 onBack만 전달.
페이지 컴포넌트 Header 사용 업데이트
src/pages/ApplicationStatusPage.tsx, src/pages/AttendancePage.tsx, src/pages/CreateStudyPage.tsx, src/pages/EditStudyPage.tsx, src/pages/StudyDetailPage.tsx, src/pages/StudyListPage.tsx, src/pages/StudyMembersPage.tsx
모든 페이지 컴포넌트에서 Header 호출 시 title prop 제거. Header가 title 없이 onBack만으로 렌더링되도록 변경. ForbiddenPage 사용 시에도 title prop 제거.
🚥 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 PR 제목이 변경 사항의 핵심을 명확하게 설명합니다. Header 컴포넌트에서 title prop 제거 및 AEGIS 텍스트 추가와 같은 주요 변경 사항을 적절히 요약하고 있습니다.

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


No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
src/components/ui/Header.tsx (1)

25-30: 인라인 스타일 대신 Tailwind arbitrary value 사용을 고려하세요.

다른 스타일은 모두 Tailwind 클래스를 사용하고 있으므로, 일관성을 위해 fontFamily 인라인 스타일을 Tailwind arbitrary value로 대체할 수 있습니다.

♻️ 수정 제안
-                <span
-                    className="font-bold text-2xl text-gray-900"
-                    style={{ fontFamily: '"Stack Sans Notch", sans-serif' }}
-                >
+                <span
+                    className="font-bold text-2xl text-gray-900 font-[Stack_Sans_Notch,sans-serif]"
+                >
src/index.css (1)

5-15: CDN URL에서 @latest 대신 특정 버전을 고정하세요.

@latest를 사용하면 CDN에서 제공하는 폰트 파일이 예고 없이 변경될 수 있어 재현 불가능한 빌드와 예상치 못한 렌더링 변화가 발생할 수 있습니다. Fontsource 공식 문서에서도 @latest 사용을 권장하지 않으며, 특정 버전으로 고정할 것을 강력히 권장합니다. 현재 stack-sans-notch의 최신 버전은 5.2.1입니다.

♻️ 수정 제안
     src:
-        url("https://cdn.jsdelivr.net/fontsource/fonts/stack-sans-notch@latest/latin-700-normal.woff2")
+        url("https://cdn.jsdelivr.net/fontsource/fonts/stack-sans-notch@5.2.1/latin-700-normal.woff2")
         format("woff2"),
-        url("https://cdn.jsdelivr.net/fontsource/fonts/stack-sans-notch@latest/latin-700-normal.woff")
+        url("https://cdn.jsdelivr.net/fontsource/fonts/stack-sans-notch@5.2.1/latin-700-normal.woff")
         format("woff");

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

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

@KwonDeaGeun KwonDeaGeun merged commit 5e64106 into develop Feb 8, 2026
2 checks passed
KwonDeaGeun added a commit that referenced this pull request Feb 8, 2026
* refactor: 언어 설정 변경 (#73)

* refactor/study list UI (#75)

* refactor: 언어 설정 변경

* refactor: 스터디 목록 최소 너비 설정

* refactor: 모바일 UI 수정

* refactor

* feat: 구글 애널리틱스 추가 (#77)

* fix: 주소창으로 권한이 없는 스터디에 접근 차단 (#80)

* fix: 주소창으로 권한이 없는 스터디에 접근 차단

* feat: 권한 없이 접근 시 에러 페이지 및 애니메이션 추가

* style: lint check 처리

* refactor: 로딩 처리 개선

* refactor: id 가드 덮어쓰기 방지

* fix: 스터디 지원 상태에 대해서 지원서 심사만 조회하도록 수정 (#82)

* fix: 스터디 지원 상태에 대해서 지원서 심사만 조회하도록 수정

* refactor: 불필요한 코드 제거

* fix: 엔드포인트 오타 수정 (#84)

* refactor: 선착순에 대해서는 지원 현황 제공하지 않도록 수정 (#86)

* refactor: 401 에러 핸들링 추가 (#88)

* refactor: 401 에러 핸들링 추가

* chore: zustand 설치

* refactor: zustand 도입 및 401 에러 핸들링

* fix: 빌드 에러 수정

* feat(STUDY-291): 스터디장 출석 현황 조회 추가 (#90)

* feat: 스터디장 출석 현황 조회 추가

* refactor: 세션 길이 0일 때 처리 및 렌더링 이슈 개선

* refactor: 불필요한 코드 제거

* fix: 로그인 화면 애니메이션 클린 수정 (#92)

* feat: 확인 여부 모달 추가 (#94)

* feat: 확인 여부 모달 추가

* refactor: 하드코딩된 부분 상속 받도록 수정 및 엔터키 기본제출 방지

* refactor: 유효성 검증 우회 방지

* fix: 수정 모드일 때 보이는 개설 텍스트를 수정으로 정정 (#95)

* refactor: 확인 여부 모달에 마진 추가 (#97)

* chore: 리드미 업데이트 (#99)

* chore: 리드미 업데이트

* refactor: 가독성 향상

* Revise project description in README

Updated project description to specify affiliation with Aegis.

* Improve clone command with clickable repository link

Updated the clone command in the README to include a clickable link.

* Fix clone command syntax in README

* Update README.md

* Update README with images and project description

Added images and enhanced project overview in README.

* Update README with additional study page images

* Add team member introduction to README

Added team member introduction section with images.

* fix: 스터디장 권한일 때 스터디 상세 페이지 타이틀 텍스트 적절하게 수정 (#100)

* refactor: 선착순 스터디에 대해서 지원현황 접근 제한 (#102)

* refactor: 선착순 스터디에 대해서 지원현황 접근 제한

* style: lint check

* fix: 승인 받고 스터디원이 아닌 경우 방지 (#105)

* fix: 권한 체크 api 경로 정정 (#107)

* fix: 권한 체크 api 경로 정정

* fix: 사용자의 권한 상태에 따라 페이지 접근 권한 처리

* chore: 리액트 돔 관련 보안 취약점 업데이트 (#110)

* fix: 로그인 버튼 문구 수정 (#112)

* fix: 로그인 버튼 문구 수정

* refactor: 로그인 페이지 내 매직넘버 상수 처리 둥 코드 리펙토링

* chore: 의존성 업데이트

* chore: vite 버전 업데이트 (#114)

* chore: vite 버전 업데이트

* style: lint check

* fix: 의존성 업데이트 후 lint fix

* fix: lint error fix

* style: 불필요한 주석 제거

* refactor(STUDY-328): 인앱 브라우저 접근성 개선 (#116)

* chore: npm 업데이트

* feat: 인앱 브라우저 리디렉션 페이지 추가

* refactor: 타이틀 타이포그라피로 변경 및 각 페이지 title 상속 제거 (#118)

---------

Co-authored-by: Yun Seongmin <ilcm96@gmail.com>
@KwonDeaGeun KwonDeaGeun deleted the refactor/unity-title-AEGIS-typography branch February 8, 2026 13:56
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.

1 participant