Skip to content

Conversation

@yunheeec
Copy link

πŸ’Ž 과제 κ΅¬ν˜„ μ„€λͺ…

과제λ₯Ό μœ„ν•΄ μ‚¬μš©ν–ˆλ˜ ν•¨μˆ˜, μˆ˜μ •ν•œ 파일 λ“± κ΅¬ν˜„ λ‚΄μš©μ— λŒ€ν•œ κ°„λ‹¨ν•œ μ„€λͺ…을 μž‘μ„±ν•΄ μ£Όμ„Έμš”.

🏁 PR 체크리슀트

  • [γ…‡] μ½”λ“œκ°€ 였λ₯˜ 없이 μ •μƒμ μœΌλ‘œ μ‹€ν–‰λ˜λ‚˜μš”?
  • [γ…‡] 컀밋 λ©”μ‹œμ§€ μ»¨λ²€μ…˜(ν…œν”Œλ¦Ώ ν™œμš©)을 μ€€μˆ˜ν–ˆλ‚˜μš”?
  • [γ…‡] 과제 λ§ˆκ°κΈ°ν•œμ„ μ€€μˆ˜ν–ˆλ‚˜μš”?

πŸ–ΌοΈ Screenshot / Video

vote history
vote page

πŸ™Œ Issue

과제 μˆ˜ν–‰ 쀑 μ–΄λ €μ› λ˜ λΆ€λΆ„μ΄λ‚˜ κΆκΈˆν–ˆλ˜ 점을 자유둭게 μž‘μ„±ν•΄ μ£Όμ„Έμš”.

Copy link

@unknownburphy unknownburphy left a comment

Choose a reason for hiding this comment

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

μ „μ²΄μ μœΌλ‘œ μ•„μ£Ό 잘 κ΅¬ν˜„ν•˜μ‹  것 κ°™μŠ΅λ‹ˆλ‹€. λ¬΄μ„œμš΄ 재λŠ₯μ΄μ‹œκ΅°μš”... LGTM!

className={`object-cover w-full h-full border-[3px] border-[#FF6841] rounded-xl
### FILL ME ###
className={`object-cover w-full h-full border-[3px] rounded-xl
${img.value === 1 ? "border-[#ff4141]" : "border-[#3457e4]"}

Choose a reason for hiding this comment

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

μ‚Όν•­μ—°μ‚°μžλ₯Ό μ΄μš©ν•΄ 잘 μ²˜λ¦¬ν–ˆλ„€μš” κ΅Ώ!

Comment on lines +70 to +81
const handleThumbsUpHover = () => {
setThumbsUpImage(require("../assets/images/thumbs-up-click.png"));
};
const handleThumbsUpLeave = () => {
setThumbsUpImage(require("../assets/images/thumbs-up-icon.png"));
};
const handleThumbsDownHover = () => {
setThumbsDownImage(require("../assets/images/thumbs-down-click.png"));
};
const handleThumbsDownLeave = () => {
setThumbsDownImage(require("../assets/images/thumbs-down-icon.png"));
};

Choose a reason for hiding this comment

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

ν•¨μˆ˜λͺ…도 μ•„μ£Ό 잘 지은 것 κ°™μŠ΅λ‹ˆλ‹€.

Copy link

@minseo25 minseo25 left a comment

Choose a reason for hiding this comment

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

μ „μ²΄μ μœΌλ‘œ κΉ”λ”ν•˜κ²Œ μ½”λ“œ 잘 μ§œμ‹  것 κ°™μ•„μš”~!

// ### thumbsUpImage Event ###
onMouseOver={handleThumbsUpHover}
onMouseOut={handleThumbsUpLeave}
onClick={() => vote(1)}
Copy link

Choose a reason for hiding this comment

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

μ €λŠ” νˆ¬ν‘œ μ‹œ vote(1); 후에 getImage(); ν˜ΈμΆœμ„ 톡해 λ‹€μŒ νˆ¬ν‘œν•  이미지λ₯Ό λΆˆλŸ¬μ™”λŠ”λ°
개인 κ΅¬ν˜„ 차이 κ°™μ•„μ„œ 참고만 ν•˜μ…”λ„ 쒋을 것 κ°™μ•„μš”~

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.

5 participants