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

feat: favicon 이미지 및 favicon 기능 #266

Merged
merged 1 commit into from
Dec 4, 2023
Merged

Conversation

khakhiD
Copy link
Contributor

@khakhiD khakhiD commented Dec 4, 2023

💻 스크린샷 (선택사항)

image

image

📃 PR 설명

favicon 이미지

favicon

vite-plugin-favicons-inject

  • https://www.npmjs.com/package/vite-plugin-favicons-inject?activeTab=readme
  • vite 빌드 도중에 favicon을 만들어서 inject 해주는 라이브러리입니다.
  • 의존성이 추가되었다는 단점이 있습니다만, 가볍고 간단합니다.
  • 매우 다양한 뷰포트를 대응해주고, favicon 이미지 사이즈도 알아서 맞춰줍니다.
  • vite.config.ts 설정 파일에 플러그인을 넣어주는 방식으로 사용합니다.

제거된 항목

  • 기존 vite 프로젝트의 기본 favicon인 public/vite.svg 아이콘을 삭제했습니다.
  • 마찬가지로 index.html head에 있는 기본 vite favicon을 적용하는 코드 라인도 제거했습니다.

📃 참고사항

🔎 PR포인트 및 궁금한 점

@khakhiD khakhiD added the 🌟 Feat 새로운 기능을 추가할 경우 label Dec 4, 2023
@khakhiD khakhiD self-assigned this Dec 4, 2023
Copy link
Member

@leeminhee119 leeminhee119 left a comment

Choose a reason for hiding this comment

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

좋습니다

@leeminhee119 leeminhee119 merged commit 98e6dca into main Dec 4, 2023
1 check passed
@leeminhee119 leeminhee119 deleted the feat/add-favicon branch December 4, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌟 Feat 새로운 기능을 추가할 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants