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

#27 emotion 사용한 기본 컴포넌트 생성 #18

Closed
wants to merge 8 commits into from

Conversation

jun2077681
Copy link
Collaborator

Summary

  • emotion의 추가에 따른 기본 컴포넌트 세팅

Describe your changes

  • 기본 div로 사용할 Box Component 작성 및 custom style 적용
  • styled-system을 사용하여 일부 테마 등 적용
    • 기존 import를 통해 사용한 color 등을 이제 string으로 바로 작성가능 (Header.tsx, Footer.tsx color 사용 부분 등)
  • header, footer 일부 에러나는 부분 emotion으로 재작성
  • 기타 코드 정리

Issue ID and link

@nix6839
Copy link
Contributor

nix6839 commented Oct 7, 2023

styled-system이 2년 넘게 유지관리가 되고 있지 않은 것 같은데 사용해도 괜찮을까요? 회의록을 보니 stitches도 같은 이유로 반려됐던 것 같습니다.

styled-system의 공식문서 사이트도 현재 접속시 그냥 GitHub 프로젝트 링크로 리다이렉트 시켜서 처음 사용하는 입장에선 GitHub의 쌩 마크다운을 찾아가면서 문서를 읽어야 되니 불편하네요. https://styled-system.com/

이 이슈에 제안된 다른 라이브러리를 사용하는 게 좋은 선택 같아 보여요.

@nix6839 nix6839 mentioned this pull request Oct 7, 2023
@jun2077681
Copy link
Collaborator Author

styled-system이 2년 넘게 유지관리가 되고 있지 않은 것 같은데 사용해도 괜찮을까요? 회의록을 보니 stitches도 같은 이유로 반려됐던 것 같습니다.

styled-system의 공식문서 사이트도 현재 접속시 그냥 GitHub 프로젝트 링크로 리다이렉트 시켜서 처음 사용하는 입장에선 GitHub의 쌩 마크다운을 찾아가면서 문서를 읽어야 되니 불편하네요. https://styled-system.com/

이 이슈에 제안된 것 다른 라이브러리를 사용하는 게 좋은 선택 같아 보여요.

styled system을 이 작업 이후로 건드릴 예정이 없었어서 이런 이슈를 몰랐네요... 링크 보내주신거 검토해볼게요 감사합니다 :)

@jun2077681 jun2077681 closed this Oct 9, 2023
@jun2077681 jun2077681 deleted the feature/emotion-base-components branch October 9, 2023 22:59
@jun2077681 jun2077681 mentioned this pull request Oct 9, 2023
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.

None yet

2 participants