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

Style: 로컬폰트, 공용 색상, 공용 폰트스타일 추가 #20

Merged
merged 10 commits into from
Feb 8, 2024

Conversation

seoyoung-min
Copy link
Contributor

@seoyoung-min seoyoung-min commented Feb 7, 2024

개요

  • url 변경에 따른 앱 구조 변경을 하였습니다.
  • 공용으로 사용할 로컬 폰트, 색상, 폰트 설정을 진행했습니다.
  • 리스트 생성 - 아이템 추가 - '아이템 추가' 버튼시 에러뜨는 오류를 살짝 껴서 함께 PR 날립니다.
  • 토스트 메시지를 위한 상수파일을 만들었습니다.

작업 사항

  • app/create -> app/list/create 폴더 구조 변경
  • 로컬 폰트 적용 (pretendard)
  • 공용 색상 globalTheme 설정
  • 공용 폰트 style 파일 생성 (font.css.ts)
  • 토스트 메시지 상수화

참고 사항 (optional)

공용 색상 사용법

  • 스타일 파일에서 vars를 import한다. import { vars } from '@/styles/theme.css'
  • 색상 넣는 칸에서 원하는 색을 입력한다. backgroundColor: vars.color.색상명,
  • 참고로 색상 명은 피그마 파일 component에 Color부분에 작성해두었습니다.
Screenshot 2024-02-07 at 19 56 56 Screenshot 2024-02-07 at 19 57 57

공용 폰트 사용법

  • 폰트 스타일은 fontSize, fontWeight, lineHeight, letterSpacing 모두 함께 이용하면 좋을 것 같아, 각 스타일을 만들어 styles/font.css 파일에 넣어두었습니다.

  • 스타일 파일에서 원하는 폰트 스타일을 가져온다. import { body1 } form '@/styles/font.css'

  • 스타일링 할때 배열로 묶어 사용한다. const button = styles([body1, {cursor: 'pointer}])

  • 로컬 폰트 적용이 계속 안돼서 시간이 오래걸렸습니다. 빠르게 전달 드리고 싶었는데...🥹

    • 로컬폰트 파일을 public에 위치시키고, 이를 기준으로 위치 경로를 작성해주니 됐습니다.
    • network 탭에서 로컬폰트 파일 다운로드 받는 것 까지 확인했습니다.
    • @fontface 이용했을 때는 사용자 로컬에 해당 폰트가 이미 있다면 다운 안받게 하는 local()와, 없을 경우 다운로드 할 경로인 url()을 함께 넣어줄 수 있었는데, vanilla extract에서는 둘 모두 넣으려 하니 적용이 안 돼 url 하나만 넣어두었습니다.

리뷰어에게

  • 공용폰트는 저것이 최선인지 확신은 없습니다.🥲 중복 코드를 최대한 작성 안하는 방안 이라고 생각했습니다!
  • 각자 담당한 것은 직접 수정하는 게 좋을 것 같아, 제가 담당했던 item 부분 + _공용 컴포넌트_에만 공용폰트, 색상을 적용했습니다.
  • @kanglocal 현지님 라이브러리 색상과 다른 것이 있어, 제가 임의로 가장 비슷한 색상들로 골랐습니다! 참고 부탁드립니다 :)
  • 토스트 메시지 상수 파일이 만들어졌습니다! 기존 토스트 메시지를 이용하셨거나 앞으로 이용하실때 txt에 들어갈 메시지는 상수로 관리해주면 좋을 것 같습니다 :)
  • app/create가 app/list/create로 경로를 바꿨습니다. 참고부탁드립니다!
  • 기존에 styles 폴더에 있던 폰트 파일을 public/fonts 폴더로 이동했습니다! pull 받으시기전에 이동시켜도 좋을 것 같습니다 :)
Screenshot 2024-02-07 at 20 06 47

@seoyoung-min seoyoung-min added the Style 코드 스타일 변경 label Feb 7, 2024
@seoyoung-min seoyoung-min self-assigned this Feb 7, 2024
Copy link

vercel bot commented Feb 7, 2024

@seoyoung-min is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pretendard라는 custom fontface를 만든 파일입니다.

boxSizing: 'border-box',
fontFamily: 'Pretendard',
fontFamily: Pretendard,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

만들어 둔 fontface를 globalStyle의 fontFamily로 적용하여, 전체 파일에 적용되도록 했습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

피그마의 라이브러리, component를 참고하여, 저희가 사용하는 폰트 스타일을 모아두었습니다.
혹시 지금 적용한 폰트의 이름이 무엇인지 궁금하시다면 피그마를 참고하시거나, 이 파일의 fontSize로 비교하시면 선택할 수 있습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

회색 색상은 숫자로 구분했는데, 홀수만 적은 이유는 혹시 중간 또는 앞 뒤로 새로운 색상이 추가 될 경우를 대비함 입니다!

Comment on lines 46 to 52
export const description = style({
color: 'gray',
color: vars.color.gray9,
});

export const url = style({
color: 'lightgray',
color: vars.color.gray7,
});
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kanglocal 저희 공용색상에 없는 색상이라, 라이브러리 내 색상 중 가장 유사한 것으로 대체했습니다! 아직 적용된 모습은 확인하지 못했습니다ㅠ 확인해주시면 감사하겠습니다!!

@@ -1,5 +1,5 @@
import { useQuery } from '@tanstack/react-query';
import * as styles from '@/components/LinkPreview/style.css';
import * as styles from '@/components/LinkPreview/LinkPreview.css';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kanglocal 저희 스타일 파일은 컴포넌트 명과 동일하게 가져가기로 한 것 같아, 파일명을 변경했습니다~! 참고 부탁드려요🙇‍♀️

@@ -1,5 +1,5 @@
import { useState, useEffect } from 'react';
import * as styles from '@/components/VideoEmbed/style.css';
import * as styles from '@/components/VideoEmbed/VideoEmbed.css';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kanglocal VideoEmbed스타일 파일명도 변경했습니다!

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

@ParkSohyunee ParkSohyunee left a comment

Choose a reason for hiding this comment

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

서영님 글로벌 스타일 적용하시느라 너무 고생많으셨습니다!🥹
폰트, 색상, 테마(body1 등) 모두 최선으로 적용된 것 같습니다!!🌈
공통 스타일 적용하니 바닐라 extract를 더 잘 활용할 수 있을 것 같네요.👍
큰 부분 맡아서 해주셔서 감사합니다.❤️‍🔥

Comment on lines +28 to +37
export const button = styleVariants({
primary: [
baseButton,
{
backgroundColor: vars.color.blue,
color: vars.color.white,
},
],
secondary: [
baseButton,
Copy link
Contributor

Choose a reason for hiding this comment

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

👍👍

Copy link
Contributor

@Nahyun-Kang Nahyun-Kang left a comment

Choose a reason for hiding this comment

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

서영님께서 스타일 공통화 시켜주시고 사용법까지 알려주셔서 편하게 사용할 수 있을 것 같습니다!🙇‍♀️정말 감사합니다!! LGTM!👍

@seoyoung-min seoyoung-min merged commit 41e40ac into 8-Sprinters:dev Feb 8, 2024
1 of 2 checks passed
@seoyoung-min seoyoung-min deleted the feature/common-style branch June 3, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Style 코드 스타일 변경
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants