Skip to content

[REFACTOR] 너비 변경#135

Merged
JangYEhoon00 merged 1 commit intomainfrom
refactor/#134-progressBar
Feb 20, 2026
Merged

[REFACTOR] 너비 변경#135
JangYEhoon00 merged 1 commit intomainfrom
refactor/#134-progressBar

Conversation

@JangYEhoon00
Copy link
Copy Markdown
Collaborator

@JangYEhoon00 JangYEhoon00 commented Feb 10, 2026

🔥 연관 이슈

🚀 작업 내용

프로그레스바의 너비를 변경했습니다
고정된 너비 -> full

🤔 고민했던 내용

💬 리뷰 중점사항

죄송하게도 해당 브랜치를 삭제한 다음 다시 작업을 하게되어 PR을 다시 올리게 됐습니다.
바쁘시겠지만 빠르게 어푸르브 해주시면 감사하겠습니다.

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 진행률 표시 바의 너비를 고정 크기에서 동적 전체 너비로 조정하여 다양한 화면 크기에서 더 나은 적응형 레이아웃을 제공합니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 10, 2026

Walkthrough

프로그레스바 컴포넌트의 컨테이너 너비를 고정값 16.5rem에서 유동 너비(w-full)로 변경했습니다. 퍼센트 고정, 색상, 내부 바 스타일링 및 전환 로직은 그대로 유지되었습니다.

Changes

Cohort / File(s) Summary
Progress Bar 스타일 수정
src/shared/ui/ProgressBar/ProgressBar.tsx
컨테이너 div의 width를 고정값 16.5rem에서 w-full(유동 너비)로 변경하여 반응형 레이아웃 지원

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • yougyung
  • ujinsim
  • keemsebin
🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 '[REFACTOR]: 너비 변경'은 변경 사항을 명확히 요약하며, 파일 요약과 일치하는 ProgressBar 컴포넌트의 너비 변경을 나타냅니다.
Description check ✅ Passed PR 설명은 필수 섹션(연관 이슈, 작업 내용)을 포함하고 있으며, 너비 변경 사항이 명확히 기술되어 있습니다.
Linked Issues check ✅ Passed 코드 변경(너비를 고정값에서 full로 변경)이 이슈 #134의 목표 요구사항을 완전히 충족합니다.
Out of Scope Changes check ✅ Passed ProgressBar 컴포넌트의 너비 변경만 수행되었으며, 모든 변경 사항이 이슈 #134의 범위 내에 있습니다.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#134-progressBar

No actionable comments were generated in the recent review. 🎉


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

Update: 2026년 02월 10일 23시 02분 38초
Storybook 미리보기: https://677a9a76c58a295e9421d3ef-nknwsmxgjh.chromatic.com/

Copy link
Copy Markdown
Collaborator

@ujinsim ujinsim left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!!!

@JangYEhoon00 JangYEhoon00 changed the title [REFACTOR]: 너비 변경 [REFACTOR] 너비 변경 Feb 11, 2026
@JangYEhoon00 JangYEhoon00 removed the request for review from yougyung February 11, 2026 15:32
@JangYEhoon00 JangYEhoon00 merged commit cd6efdc into main Feb 20, 2026
11 checks passed
@JangYEhoon00 JangYEhoon00 deleted the refactor/#134-progressBar branch February 20, 2026 04:59
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.

[REFACTOR] 프로그레스바 너비 변경

2 participants