Skip to content

스타일 관련 수정사항#472

Merged
enunsnv merged 8 commits intomainfrom
fix/style
Mar 14, 2026
Merged

스타일 관련 수정사항#472
enunsnv merged 8 commits intomainfrom
fix/style

Conversation

@enunsnv
Copy link
Contributor

@enunsnv enunsnv commented Mar 12, 2026

작업 내용

수정 전 수정 후
image 스크린샷 2026-03-13 01 14 49

마이 페이지에서 회원 유형의 폰트의 크기를 회원 유형에 무관하게 나타나도록 수정하였습니다.

리뷰 요구사항 (선택)

서현님께서 남겨주신 리뷰 중 이미지 1이미지 3의 문제는 이미 해결되어있는 것으로 확인하였고, 이미지 2에서 1번 이슈를 해결한 내용이 해당 pr 입니다! 이미지 2에서 2번째 이슈인 아이콘 정렬 관련 문제를 이어서 해결해보도록 하겠습니다!

@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connect-web-admin Ready Ready Preview, Comment Mar 12, 2026 4:18pm
solid-connection-web Ready Ready Preview, Comment Mar 12, 2026 4:18pm

@github-actions github-actions bot added the web label Mar 12, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 12, 2026

Walkthrough

이 변경사항들은 애플리케이션의 스타일 및 색상 시스템에 관한 업데이트입니다. 다음과 같이 구성되어 있습니다:

  1. MyProfileContent 컴포넌트 - 사용자 역할 텍스트의 타이포그래피 스타일을 조건부 적용에서 고정된 클래스 세트로 변경하였습니다.

  2. SearchBar 컴포넌트 - 입력 필드의 테두리 색상을 회색 계열에서 새로운 회색-C 계열로 업데이트했습니다.

  3. 전역 스타일 - .score-table 테이블 스타일 내에서 Tailwind 유틸리티 클래스의 순서를 조정했습니다.

  4. Tailwind 설정 - gray-c 색상 팔레트에 200과 300 톤의 새로운 색상 밸류를 추가하여 사용 가능한 색상 옵션을 확대했습니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • wibaek
  • manNomi
  • khwww
🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (3 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive PR 제목이 '스타일 관련 수정사항'으로 너무 일반적이어서 구체적인 변경 내용을 파악하기 어렵습니다. 제목을 '마이 페이지 회원 유형 폰트 크기 통일' 또는 유사한 구체적인 표현으로 변경하여 주요 변경 내용을 명확히 해주세요.
Linked Issues check ❓ Inconclusive PR은 마이 페이지 폰트 크기 수정에 집중했지만, 링크된 이슈 #369는 '지원자 현황 페이지의 에러 토스트' 문제를 다루므로 직접적인 연관성이 불명확합니다.
Description check ❓ Inconclusive PR 설명이 작업 내용 섹션에서 마이페이지 회원 유형 폰트 크기 수정에 대해 명확히 설명하고 비포/애프터 이미지를 포함했으나, 관련 이슈 번호와 특이 사항 섹션이 누락되어 있습니다. 관련 이슈 섹션에 'resolves: #369'를 추가하고, 다른 스타일 변경사항(검색바 테두리색, 색상 팔레트 확장 등)을 특이 사항 섹션에서 설명해 주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Out of Scope Changes check ✅ Passed 변경사항들(타이포그래피, 색상 추가, 테두리 색상)이 모두 스타일 관련 수정으로 일관성 있게 PR 목표와 일치합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/style
📝 Coding Plan for PR comments
  • Generate coding plan

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
apps/web/tailwind.config.ts (1)

150-154: 2. gray-c shade 번호만 한번 맞춰주세요.

Line 153의 gray-c-300 값인 #f9fafb100/200보다 더 밝아서 현재 scale이 역순입니다. 지금은 문제 없더라도 나중에 숫자만 보고 tone을 고르면 의도와 다르게 적용되기 쉬우니, 값 자체를 더 어둡게 조정하거나 shade 번호를 재배치하는 편이 안전합니다.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@apps/web/tailwind.config.ts` around lines 150 - 154, The "gray-c" color scale
in tailwind.config.ts has an out-of-order tone: the "300" value "#f9fafb" is
lighter than "100" and "200", so reorder or adjust values so numeric keys
increase in darkness; update the "gray-c" object (keys "100","200","300") so
"300" is a darker hex than "200" (or swap the hexes and keep keys consistent) to
ensure the scale progresses from light (100) to dark (300).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@apps/web/tailwind.config.ts`:
- Around line 150-154: The "gray-c" color scale in tailwind.config.ts has an
out-of-order tone: the "300" value "#f9fafb" is lighter than "100" and "200", so
reorder or adjust values so numeric keys increase in darkness; update the
"gray-c" object (keys "100","200","300") so "300" is a darker hex than "200" (or
swap the hexes and keep keys consistent) to ensure the scale progresses from
light (100) to dark (300).

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: e4ef314a-0807-48bd-a79b-ab409ede7b82

📥 Commits

Reviewing files that changed from the base of the PR and between 8472772 and c282cda.

📒 Files selected for processing (4)
  • apps/web/src/app/my/_ui/MyProfileContent/index.tsx
  • apps/web/src/app/university/search/SearchBar.tsx
  • apps/web/src/styles/globals.css
  • apps/web/tailwind.config.ts

Copy link
Contributor

@manNomi manNomi left a comment

Choose a reason for hiding this comment

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

너무 좋아용 은정님 최고!!

@enunsnv enunsnv merged commit 527556c into main Mar 14, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants