Skip to content
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

[MOYEO-68] 볼륨 on/off 아이콘 추가 & 리드미 수정 #46

Merged
merged 4 commits into from
Aug 21, 2023

Conversation

haryung-lee
Copy link
Member

@haryung-lee haryung-lee commented Aug 20, 2023

작업 내용

-볼륨 on/off 아이콘 추가
-readme에 캐릭터 추가

체크리스트

  • Code Review 요청
  • Label 설정
  • PR 제목 규칙에 맞는지 확인

@haryung-lee haryung-lee added the 🎨 Style 스타일링 label Aug 20, 2023
@haryung-lee haryung-lee self-assigned this Aug 20, 2023
@haryung-lee haryung-lee added ✨ Feature 기능 추가 and removed 🎨 Style 스타일링 labels Aug 20, 2023
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 20, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 685d86e
Status: ✅  Deploy successful!
Preview URL: https://0554ca5d.moyeomoyeo.pages.dev
Branch Preview URL: https://feature-moyeo-68.moyeomoyeo.pages.dev

View logs

@haryung-lee haryung-lee changed the title [MOYEO-68] 볼륨 on/off 아이콘 추가 [MOYEO-68] 볼륨 on/off 아이콘 추가 & 리드미 수정 Aug 20, 2023
Copy link
Collaborator

@young-do young-do left a comment

Choose a reason for hiding this comment

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

👍 👍

README.md Outdated
</tr>
</tbody>
</table>
<a href="https://forms.gle/SScnNAsaziQWfCi89" style="font-style:italic; font-size:18px; color:#8060FF">귀여운 캐릭터들 이름 추천 받아요~!</a>
Copy link
Collaborator

Choose a reason for hiding this comment

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

style 속성이 지워지는듯! i 태그를 감싸는건 어때?

Comment on lines +81 to +85
{isPlayedBackgroundMusic ? (
<VolumeOnIcon width="25px" height="25px" />
) : (
<VolumeOffIcon width="25px" height="25px" />
)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이렇게 하는것도 괜찮다~!
나중에 svg width, height 전달하는거 통일하면 좋을 듯 😄

@haryung-lee haryung-lee merged commit b3df819 into main Aug 21, 2023
1 check passed
@haryung-lee haryung-lee deleted the feature/MOYEO-68 branch August 21, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants