-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
π Style: μ νλ νκ·Έ μ΄λνλ μ λλ©μ΄μ ꡬν μμ± #79
Conversation
Caution Review failedThe pull request is closed. π WalkthroughWalkthroughμ΄λ² PRμ μ μ CSSμμ μ λλ©μ΄μ κ΄λ ¨ μ£Όμ μ κ±° λ° λΆνμν μμΉ μ§μ ν΄λμ€λ€μ μμ νκ³ , λ©ν° μ€ν νΌ λ΄ νκ·Έ κ΅¬μ± νμΌμμ νκ·Έμ μμΉ κ°μ μ‘°μ νλ ννΈ, νκ·Έ UI μ»΄ν¬λνΈλ€μ λν΄ ν΄λ¦ μ΄λ²€νΈ νΈλ€λ¬ μΆκ°μ λμ μμΉ μ€μ λ‘μ§μ λμ νμμ΅λλ€. Changes
Sequence Diagram(s)sequenceDiagram
participant μ¬μ©μ
participant TagBox
participant Tag
μ¬μ©μκ°->>Tag: νκ·Έ ν΄λ¦ μ΄λ²€νΈ λ°μ
Tag-->>TagBox: ν΄λ¦ μ΄λ²€νΈ μ λ¬ (onClick νΈλ€λ¬)
TagBox->>TagBox: getTagPositions ν¨μ μ€ννμ¬ νκ·Έ μμΉ κ²°μ
TagBox-->>μ¬μ©μ: μλ‘μ΄ νκ·Έ μμΉ μ
λ°μ΄νΈ λ°μ
Possibly related PRs
Suggested labels
Suggested reviewers
Tip β‘π§ͺ Multi-step agentic review comment chat (experimental)
π Recent review detailsConfiguration used: .coderabbit.yml π Files selected for processing (2)
β¨ Finishing Touches
πͺ§ TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
π§Ή Nitpick comments (2)
src/features/multi-step-form/ui/careerForm/tagFormStep/ui/TagBox.tsx (2)
31-40
: νκ·Έ κ³ μ μμΉ μμλ₯Ό μ μνμ΅λλ€.4κ° μ΄μμ νκ·Έκ° μ νλ λ κ° νκ·Έμ κ³ μ μμΉλ₯Ό μ μνλ μμκ° μΆκ°λμμ΅λλ€. μ΄λ νκ·Έ μ ν μ νΉμ μμΉλ‘ μ΄λνλ μ λλ©μ΄μ μ ꡬννκΈ° μν μ’μ μ κ·Όλ°©μμ λλ€.
μΆνμ μ΄ μμλ₯Ό λ³λμ νμΌλ‘ λΆλ¦¬νλ κ²μ΄ μ’μ κ² κ°μ΅λλ€. μ£Όμμλ μΈκΈλμ΄ μλ―μ΄ νμΌ λΆλ¦¬λ₯Ό κ³ λ €ν΄λ³΄μΈμ.
42-78
: νκ·Έ μμΉ κ³μ° ν¨μκ° μΆκ°λμμ΅λλ€.μ νλ νκ·Έμ μμΉλ₯Ό κ³μ°νλ getTagPositions ν¨μκ° μΆκ°λμμ΅λλ€. μ νλ νκ·Έ μμ μ ν μμμ λ°λΌ νκ·Έμ μμΉλ₯Ό λμ μΌλ‘ κ²°μ ν©λλ€.
ν¨μκ° λ€μ 볡μ‘νκ³ μ‘°κ±΄λ¬Έμ΄ λ§μ κ°λ μ±μ΄ λ¨μ΄μ§λλ€. λ€μκ³Ό κ°μ΄ 리ν©ν λ§μ κ³ λ €ν΄λ³΄μΈμ:
function getTagPositions(position: string, tag: TagConfigItem) { // 4κ° μ΄μ μ νλ κ²½μ° κ³ μ μμΉ λ°ν if (tagArray.length >= 4 && tagArray.includes(tag.message)) { return tagFixedPositions[tag.message]; } + // μ νλ νκ·Έμ μΈλ±μ€ μ°ΎκΈ° + const tagIndex = tagArray.indexOf(tag.message); + if (tagIndex === -1) return position; + // μμΉ νμ νμ + const isTop = position.includes('top'); + const isBottom = position.includes('bottom'); + const isLeft = position.includes('left'); + const isRight = position.includes('right'); + // μμΉμ λ°λ₯Έ μ’ν λ°ν + if (tagIndex === 0) { + if (isTop && isLeft) return 'left-[calc(50%-40px)] top-[calc(50%-56px)]'; + if (isTop && isRight) return 'right-[calc(50%-28px)] top-[calc(50%-56px)]'; + if (isBottom && isRight) { + return tag.message === 'μ·¨λ―Έ' + ? 'right-[calc(50%-28px)] bottom-[calc(50%+16px)]' + : 'right-[calc(50%-52px)] bottom-[calc(50%+16px)]'; + } + if (isBottom && isLeft) return 'left-[calc(50%-40px)] bottom-[calc(50%+16px)]'; + } + // λλ¨Έμ§ μΈλ±μ€μ λν μ²λ¦¬... }
π Review details
Configuration used: .coderabbit.yml
Review profile: CHILL
Plan: Pro
π Files selected for processing (4)
src/app/globals.css
(1 hunks)src/features/multi-step-form/ui/careerForm/tagFormStep/config/config.ts
(2 hunks)src/features/multi-step-form/ui/careerForm/tagFormStep/ui/TagBox.tsx
(2 hunks)src/shared/ui/tag/tag.tsx
(3 hunks)
π Additional comments (12)
src/app/globals.css (1)
165-165
: μ£Όμλ§ μ κ±°λμκ³ κΈ°λ₯μ λ³νλ μμ΅λλ€.CSS μ λλ©μ΄μ μ transform μμ±μμ μ£Όμμ΄ μ κ±°λμμ§λ§, μ½λμ κΈ°λ₯μ λ³κ²½λμ§ μμμ΅λλ€.
src/shared/ui/tag/tag.tsx (3)
18-18
: Tag μ»΄ν¬λνΈμ ν΄λ¦ μ΄λ²€νΈ μ²λ¦¬ κΈ°λ₯μ΄ μΆκ°λμμ΅λλ€.Tag μ»΄ν¬λνΈμ onClick μμ±μ μΆκ°νμ¬ ν΄λ¦ μ΄λ²€νΈλ₯Ό μ²λ¦¬ν μ μλ κΈ°λ₯μ΄ μΆκ°λμμ΅λλ€. μ΄κ²μ νκ·Έ μ ν μ λλ©μ΄μ ꡬνμ μν μ’μ λ³κ²½μ¬νμ λλ€.
41-41
: ν¨μ μκ·Έλμ²κ° μ λ°μ΄νΈλμμ΅λλ€.onClick 맀κ°λ³μκ° Tag ν¨μ μκ·Έλμ²μ μΆκ°λμμ΅λλ€. μ΄λ‘μ¨ ν΄λ¦ μ΄λ²€νΈ μ²λ¦¬ κΈ°λ₯μ΄ μ»΄ν¬λνΈμ ν΅ν©λμμ΅λλ€.
63-63
: span μμμ onClick νΈλ€λ¬κ° μΆκ°λμμ΅λλ€.span μμμ onClick νΈλ€λ¬κ° μΆκ°λμ΄ Tag μ»΄ν¬λνΈκ° ν΄λ¦ μ΄λ²€νΈμ μλ΅ν μ μκ² λμμ΅λλ€.
src/features/multi-step-form/ui/careerForm/tagFormStep/config/config.ts (4)
7-7
: νμ μ μΈμμ μ£Όμμ΄ μ κ±°λμμ΅λλ€.message μμ±μ νμ μ μΈμμ μ£Όμμ΄ μ κ±°λμμ΅λλ€. κΈ°λ₯μ λ³νλ μμ΅λλ€.
54-54
: νκ·Έ μμΉκ° μ‘°μ λμμ΅λλ€.'μ΅κ·Ό μμ' νκ·Έμ μμΉκ° 'bottom-40'μμ 'bottom-40 left-0'λ‘ λ³κ²½λμμ΅λλ€. μ΄λ μ λλ©μ΄μ ꡬνμ μν μμΉ μ‘°μ μ λλ€.
62-62
: νκ·Έ μμΉκ° μ‘°μ λμμ΅λλ€.'νλ μ§μ' νκ·Έμ μμΉκ° 'top-20'μμ 'top-20 left-0'λ‘ λ³κ²½λμμ΅λλ€. μ΄λ μ λλ©μ΄μ ꡬνμ μν μμΉ μ‘°μ μ λλ€.
70-70
: νκ·Έ μμΉκ° μ‘°μ λμμ΅λλ€.'μμ μ 보' νκ·Έμ μμΉκ° 'left-1/2 -translate-x-1/2'μμ 'left-40 top-0'λ‘ λ³κ²½λμμ΅λλ€. μ΄λ μ λλ©μ΄μ ꡬνμ μν μμΉ μ‘°μ μ λλ€.
src/features/multi-step-form/ui/careerForm/tagFormStep/ui/TagBox.tsx (4)
7-7
: TagConfigItem νμ κ°μ Έμ€κΈ°κ° μΆκ°λμμ΅λλ€.TagConfigItem νμ μ λͺ μμ μΌλ‘ κ°μ Έμμ νμ μμ μ±μ ν₯μμμΌ°μ΅λλ€.
80-83
: useEffectκ° μ΄λλμμ΅λλ€.useEffect ν μ΄ μ»΄ν¬λνΈ λ λΆλΆμΌλ‘ μ΄λλμμ΅λλ€. κΈ°λ₯μ λ³νλ μμ΅λλ€.
97-97
: νκ·Έ μμΉ κ³μ° λ‘μ§μ΄ μ λ°μ΄νΈλμμ΅λλ€.νκ·Έ μμΉ κ³μ° λ‘μ§μ΄ μ λ°μ΄νΈλμ΄ μλ‘ μΆκ°λ getTagPositions ν¨μλ₯Ό μ¬μ©ν©λλ€. μ΄λ‘μ¨ μ νλ νκ·Έλ λμ μΌλ‘ κ³μ°λ μμΉλ‘ μ΄λν μ μκ² λμμ΅λλ€.
99-99
: Click μ΄λ²€νΈ νΈλ€λ¬κ° Tag μ»΄ν¬λνΈλ‘ μ΄λλμμ΅λλ€.onClick μ΄λ²€νΈ νΈλ€λ¬κ° νκ·Έ μ»΄ν¬λνΈλ‘ μ΄λλμ΄ νκ·Έ μ ν κΈ°λ₯μ΄ κ°μ λμμ΅λλ€. μ΄λ UI/UX κ°μ μ μ’μ λ³νμ λλ€.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
κ³ μνμ ¨μ΅λλ€~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
μ κΈ°μ€μμ μ΄ μ½λλ μ΄λΈνκ² μ μμ±λ κ² κ°μ΅λλ€. κ³ μνμ
¨μ΅λλ€. Approve
λλ¦¬κ² μ΅λλ€. π
π κ°μ
#76 [Feat] μ νν νκ·Έ μ΄λνλ μ λλ©μ΄μ ꡬν
π λ³κ²½μ¬ν
κΈ°λ₯
νλ©΄
β 체ν¬μ¬ν
Summary by CodeRabbit
New Features
Style
Bug Fixes