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: add favicon #99

Merged
merged 1 commit into from
Jul 3, 2024
Merged

feat: add favicon #99

merged 1 commit into from
Jul 3, 2024

Conversation

DaleSeo
Copy link
Contributor

@DaleSeo DaleSeo commented Jul 3, 2024

몇 주 전에 @nhistory 님께서 만들어주신 Favicon 파일이 아직도 추가가 안 되어 있길래 후다닥 추가합니다 :)

Checklist before merging

  • Link an issue with the pull request
  • Ensure no errors or warnings on the browser console
  • Avoid additional major pushes after approval (if necessary, request a new review)

@DaleSeo DaleSeo linked an issue Jul 3, 2024 that may be closed by this pull request
@DaleSeo DaleSeo marked this pull request as ready for review July 3, 2024 23:18
@DaleSeo DaleSeo requested a review from a team as a code owner July 3, 2024 23:18
Comment on lines -148 to +162
grid-template-columns: minmax(306px, 1fr);
grid-template-columns: minmax(306px, 1fr);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

여기는 그냥 자동 포멧팅이 된 부분입니다. 예전에 포멧팅을 일괄 정리했는데 왜 자꾸 포멧팅이 틀어지는 모르겠네요 ㅋㅋ

Copy link
Contributor

Choose a reason for hiding this comment

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

요 부분 제가 작업하면서 띄어쓰기 된 채로 푸쉬되어서 그런 것 같습니다....😅 다시 이번 pr에서 맞춰두었습니다!

Copy link
Contributor Author

@DaleSeo DaleSeo Jul 3, 2024

Choose a reason for hiding this comment

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

@yolophg 아하, 그렇군요. 그나저나 요 것이 Format on save 설정을 꺼놨을 때의 단점 아닐까요? 포멧팅 해주는 것을 까먹을 수 있다 ㅎㅎㅎ

@yolophg
Copy link
Contributor

yolophg commented Jul 3, 2024

favicon은 빠르게 머지되면 좋을 것 같습니다. 감사합니다!

@DaleSeo DaleSeo merged commit ee5575e into main Jul 3, 2024
1 check passed
@DaleSeo DaleSeo deleted the favicon branch July 3, 2024 23:57
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.

Favicon 추가
2 participants