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

[Feature] 크리스마스 보드, 폴라로이드 테마 추가, Google Tag Manager 설정 #139

Merged
merged 10 commits into from
Dec 15, 2024

Conversation

hwanheejung
Copy link
Member

@hwanheejung hwanheejung commented Dec 15, 2024

👀 관련 이슈

크리스마스

✨ 작업한 내용

  • 홈화면, 로고 크리스마스로 변경했습니다
  • 크리스마스 테마 추가했습니다
  • 다크모드 헤더일 때 글자색 하얀색으로 만들었습니다
  • 구글 태그매니저 추가했습니다

🌀 PR Point

고칠 부분이 있으면 알려주세요!

🍰 참고사항

NEXT_PUBLIC_GTM_ID 추가해주세요 디스코드에 있습니다!

📷 스크린샷 또는 GIF

홈화면 보드 테마 폴라로이드 테마
스크린샷 2024-12-15 오후 6 02 28 스크린샷 2024-12-15 오후 8 28 29 스크린샷 2024-12-15 오후 6 03 54

@hwanheejung hwanheejung self-assigned this Dec 15, 2024
@@ -13,6 +13,50 @@ export const THEMAS: Record<ThemaKeyType, ThemaType> = {
descriptionStyle:
'linear-gradient(180deg, rgba(255, 255, 255, 0.20) 10.71%, rgba(255, 255, 255, 0.50) 57.96%, rgba(255, 255, 255, 0.00) 100%), #eaeaea',
},
'F-12': {
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 순서는 lint가 자동으로 한건가요? 아니라면 0부터 순서대로되면 좋을것 같습니다!

Copy link
Member Author

@hwanheejung hwanheejung Dec 15, 2024

Choose a reason for hiding this comment

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

디자인팀에서 정해준 폴라로이드 순서가 변경돼서 이렇게 바꿨는데, 순서대로 하려면 파일 이름을 모두 변경해야 합니다. 더 좋은 방법이 있을까요??

Copy link
Collaborator

Choose a reason for hiding this comment

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

아 순서때문에 그런거군요. 순서가 중요하다면 Object.entries 대신 보여줄 테마 키값을 별도의 배열로 관리하는게 나을 것 같은데 어떨까요?

Copy link
Collaborator

@junseublim junseublim left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@hwanheejung hwanheejung merged commit 059ef7e into develop Dec 15, 2024
@hwanheejung hwanheejung deleted the feature/christmas-theme branch December 15, 2024 12:09
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.

2 participants