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] : spring theme 추가 #399 #406

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

lamPolar
Copy link
Contributor

@lamPolar lamPolar commented Apr 10, 2024

Describe your changes

  • 벚꽃 테마를 추가했습니다.
  • 기선님이 webm 형식으로 바꾸신 부분과 충돌이 걱정되어, sit.gif, sit.webm을 모두 추가해두었습니다. 추후 모두 머지 되면 sit.gif를 지워야합니다.
  • create, pseronal 페이지에 spring을 추가했습니다. (이과정에서 생각보다 건드려야할 config가 많더라구요 이것도 건드리는 부분을 하나로 모을수 있을까요??)
  • 벚꽃테마 한정 벚꽃 애니메이션을 추가했습니다.
    • 지금 약간 random값이 이상하게 되어서 꽃잎이 두개씩 같이 떨어지는 느낌인데, 이부분은 풀리퀘 머지전에 수정보겠습니다.
    • 기본적으로 snowanimation에 animation과 size만 수정을 한 버전으로 만들었는데, 최종 픽스되면 BackgroundAnimation컴포넌트로 만들어서 재사용하는 방향을 생각하고 있습니다.

Screen Capture

Screen.Recording.2024-04-10.at.5.23.59.PM.mov

Issue number and link

@lamPolar lamPolar added the ✨ feature New feature or improvement label Apr 10, 2024
@lamPolar lamPolar self-assigned this Apr 10, 2024
Copy link
Contributor

@Arkingco Arkingco left a comment

Choose a reason for hiding this comment

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

확인했습니다!
webm 작업한 브랜치가 머지되어서 main pull 받고 gif파일을 webm파일로 변경 하시면 문제 없을 것 같습니다!! 고생하셨어용

@Arkingco
Copy link
Contributor

테마 코드 추가 할 때 생기는 비용 줄이는 내용은 좀 더 생각해보고 말씀 드리겠습니다!!

Copy link
Contributor

@lyssoi lyssoi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature New feature or improvement
Projects
Status: Updated
Development

Successfully merging this pull request may close these issues.

3 participants