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 : 프로젝트 세팅 #2

Merged
merged 22 commits into from
Mar 3, 2024
Merged

feat : 프로젝트 세팅 #2

merged 22 commits into from
Mar 3, 2024

Conversation

Ubinquitous
Copy link
Member

@Ubinquitous Ubinquitous commented Mar 3, 2024

Issue Number

#4

What

프로젝트 세팅

Why

생산적인 비즈니스 속도를 위한 라이브러리 추가

How

.

ScreenShot

.

@Ubinquitous Ubinquitous requested review from 5ewon and jyh071116 March 3, 2024 14:53
@Ubinquitous Ubinquitous self-assigned this Mar 3, 2024
@Ubinquitous Ubinquitous added chore 자잘한 변경 style 스타일 변경 (퍼블리싱말고 코드 스타일) ci CI labels Mar 3, 2024
Copy link
Member

@jyh071116 jyh071116 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!
혹시 주로 사용하던 yarn 대신 pnpm으로 넘어간 이유가 있을까요?

Copy link
Member

@5ewon 5ewon left a comment

Choose a reason for hiding this comment

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

Good Good 🙌🙌

app/globals.css Outdated
@import url("https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/variable/pretendardvariable.min.css");

* {
font-family: "Pretendard Variable", Pretendard, -apple-system,
Copy link
Member

Choose a reason for hiding this comment

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

프리텐다드 가변 폰트 사용하시는 이유가 있을까요?
가변폰트의 특징인 스타일의 넓은 선택폭은 장점이기도 하지만 역으로 굳이 불필요한 폰트 변형의 여지를 남겨두기에 오히려 이런 점을 선호하지 않은 디자이너 분도 계신 것 같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

말씀해주신 이슈 확인했습니다!
Pretendard Static으로 폰트 변경 완료했습니다!

@import url("https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/static/pretendard.min.css");

* {
  font-family:
    "Pretendard",
    Pretendard,
    -apple-system,
    BlinkMacSystemFont,
    system-ui,
    Roboto,
    "Helvetica Neue",
    "Segoe UI",
    "Apple SD Gothic Neo",
    "Noto Sans KR",
    "Malgun Gothic",
    "Apple Color Emoji",
    "Segoe UI Emoji",
    "Segoe UI Symbol",
    sans-serif;
  margin: 0;
  padding: 0;
  box-sizing: border-box;
}

@Ubinquitous
Copy link
Member Author

@jyh071116 pnpm과 관련해서 말씀주셔서 감사합니다!
pnpm은 프로젝트별로 node_modules에서 매번 패키지를 설치하지 않고 global 저장소에 패키지를 한 번만 저장하여 저장공간을 절약함 뿐만이 아니라 빌드시 속도도 yarn보다 빠릅니다!
그리고 저또한 yarn을 사용하며 Ghost Dependency 이슈를 종종 겪은 상태였는데, pnpm이 이를 해결해줄 수 있어 이번 프로젝트 때에는 pnpm을 사용하기로 했습니다!!

@Ubinquitous
Copy link
Member Author

수고하셨습니다!

@Ubinquitous Ubinquitous merged commit 225bef6 into main Mar 3, 2024
Ubinquitous added a commit that referenced this pull request Mar 5, 2024
FIX : children 컴포넌트 공급 오류 수정
@Ubinquitous Ubinquitous deleted the feat/setting branch November 15, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore 자잘한 변경 ci CI style 스타일 변경 (퍼블리싱말고 코드 스타일)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants