Skip to content

fix(web): mentorPage 디자인 수정#508

Open
enunsnv wants to merge 9 commits intomainfrom
fix/style
Open

fix(web): mentorPage 디자인 수정#508
enunsnv wants to merge 9 commits intomainfrom
fix/style

Conversation

@enunsnv
Copy link
Copy Markdown
Contributor

@enunsnv enunsnv commented May 1, 2026

관련 이슈

  • 프로필 이미지 svg 에셋 Figma 디자인과 상이하여 변경 필요
  • 프로필 테두리 색상 또한 Figma 디자인과 상이하여 변경 필요
  • 피그마 시안과 동일하게 수정 필요

작업 내용

MentorPage MentorCard MenteePage
MentorPage MentorCard MenteePage
showMentorBadge ? "border-2 border-primary-2" : "border-none"

@enunsnv enunsnv requested review from manNomi and wibaek as code owners May 1, 2026 05:10
@vercel
Copy link
Copy Markdown

vercel Bot commented May 1, 2026

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

Project Deployment Actions Updated (UTC)
solid-connection-web Ready Ready Preview, Comment May 10, 2026 4:43am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped May 10, 2026 4:43am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 1b17134f-d5cd-4dd7-b0e1-92f43b2f8721

📥 Commits

Reviewing files that changed from the base of the PR and between f89cfbd and 09d25b0.

📒 Files selected for processing (1)
  • apps/web/src/components/mentor/MentorCard/index.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • apps/web/src/components/mentor/MentorCard/index.tsx

Walkthrough

  1. 문서 정리. .env.guide.md에서 AI Inspector 관련 Web 런타임 및 GitHub Actions Worker 환경 변수 등록 안내(약 39줄)를 제거했습니다.
  2. 환경 변수 추가. .env.previewNEXT_PUBLIC_COOKIE_LOGIN_ENABLED=true를 새로 추가했습니다.
  3. UI 스타일 변경. 여러 컴포넌트(ProfileWithBadge, MenteePageTabs, MentorFindSection, MyMentorSection, MentorPage, MentorCard)의 Tailwind 클래스와 일부 레이아웃을 조정했습니다.
  4. 기능·API 미변경. 위 변경들은 렌더링/스타일에 국한되며 로직이나 공개 인터페이스는 변경되지 않았습니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • wibaek
  • khwww
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 'fix(web): mentorPage 디자인 수정'은 멘토 페이지의 디자인 수정이라는 주요 변경사항을 명확하게 요약하며 변경 세트의 핵심을 잘 전달합니다.
Description check ✅ Passed PR 설명에서 관련 이슈 섹션은 작성되었으나 이슈 번호 링크가 명시되지 않았고, 작업 내용은 스크린샷과 코드 예제로 잘 설명되어 있으며 특이 사항과 리뷰 요구사항 섹션은 채워지지 않았습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 fix/style

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

@github-actions github-actions Bot added the web label May 1, 2026
@enunsnv enunsnv changed the title Fix/style fix(web): 프로필 이미지 컴포넌트 일부 스타일 디자인 시안과 동일하게 수정 May 1, 2026
@Designer-Su
Copy link
Copy Markdown

안녕하세요, 은정님 고생해주셔서 감사합니다 :)

저의 코멘트는 아래와 같습니다.

  1. 프로필 설정이 안되어있을때 기존 사람 이미지에서 K100 색상이 현재 디자인인데, 해당 디자인으로 들어가는것일까요?
  2. 카드 채널 컴포넌트를 조금 변경하였습니다. (글자간의 패딩값, 글자 색상 K900으로 변경, 채널 카드에서 합격레시피 넣기 등등 자세한 컴포넌트 변경은 아래 피그마에서 확인 부탁드립니다.

피그마 링크:https://www.figma.com/design/TGJs0mxQwRoGVsVjCOIHi7/SLCN-Design-System--App-?node-id=1597-10703&t=2HY0M7gOHRTLx3hH-1
3. 작업해주신 페이지의 패딩값이 디자인과 다른 부분이 있는 것 같습니다. 참고용으로 디자인 캡쳐와 링크 전달드립니다 :)

이미지: Screenshot 2026-05-01 at 6 07 13 PM

피그마 링크: https://www.figma.com/design/TGJs0mxQwRoGVsVjCOIHi7/SLCN-Design-System--App-?node-id=3232-41710&t=2HY0M7gOHRTLx3hH-1

  1. 멘티가 보는 페이지에서의 패딩값도 멘토 페이지의 수정 사항까지 같이 반영 부탁드립니다!

보기시 어려우신 부분있으면 언제나 미팅 요청 주세요!
감사합니다 :)

Copy link
Copy Markdown
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.

border 색상이 잘못들어가있었군요
수경님 말씀주신것만 추가로 반영해주셔도 감사할것 같습니다!!

@vercel vercel Bot temporarily deployed to Preview – solid-connect-web-admin May 10, 2026 02:37 Inactive
@enunsnv enunsnv changed the title fix(web): 프로필 이미지 컴포넌트 일부 스타일 디자인 시안과 동일하게 수정 fix(web): mentorPage 스타일 디자인 시안과 동일하게 수정 May 10, 2026
@enunsnv
Copy link
Copy Markdown
Contributor Author

enunsnv commented May 10, 2026

네 피드백 감사합니다! 해당 사항들 전부 반영하여 다시 작업하였습니다🙂

MentorPage MentorCard MenteePage
MentorPage MentorCard MenteePage

추가적으로 궁금한 사항은 다음과 같습니다🙋🏻‍♀️

현재 개발 환경에는 누적 멘토 n명 항목이 존재하는데, 피그마 시안에는 해당 부분이 없습니다.
피그마 환경과 동일하게 해당 항목을 제거하면 되는지 혹은 그대로 유지하면 되는지 궁금합니다!

확인 후 다시 피드백 주시면 감사하겠습니다!

@enunsnv enunsnv changed the title fix(web): mentorPage 스타일 디자인 시안과 동일하게 수정 fix(web): mentorPage 스타일 디자인 수정 May 10, 2026
@enunsnv enunsnv changed the title fix(web): mentorPage 스타일 디자인 수정 fix(web): mentorPage 디자인 수정 May 10, 2026
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.

3 participants