Skip to content

[CHORE] design system 도입#1

Merged
FLU0RITE merged 5 commits intodevelopfrom
chore/design_system
Apr 3, 2025
Merged

[CHORE] design system 도입#1
FLU0RITE merged 5 commits intodevelopfrom
chore/design_system

Conversation

@FLU0RITE
Copy link
Collaborator

@FLU0RITE FLU0RITE commented Apr 2, 2025

#️⃣연관된 이슈

close #

📝작업 내용

  • design system 도입
  • android 그래들 최소 sdk 설정 (23)

스크린샷 or 작업영상

💬리뷰 요구사항

@FLU0RITE FLU0RITE requested a review from kkosang April 2, 2025 14:42
@FLU0RITE FLU0RITE self-assigned this Apr 2, 2025
Copy link
Collaborator

@kkosang kkosang left a comment

Choose a reason for hiding this comment

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

굿입니다
브랜치 작업도 깔끔하게했네요 👍

- name: Analyze code
run: flutter analyze

- name: Check code formatting
Copy link
Collaborator

Choose a reason for hiding this comment

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

현재 ci에서 코드포맷은 뺀건가요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

맞습니다 ci와 버전을 맞췄음에도 불구하고

로컬에서 dart format . 을 하면 바꿀 것이 없다고 뜨는데

ci 에선 한 개가 변화가 있다고 뜨네요

줄 바꿈에 문제가 있는 것이라고 추측이 되는데 이 부분 때문에 무엇을 추가한다기 보단 일단 뺐습니다.

Copy link
Collaborator

Choose a reason for hiding this comment

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

오홍 그렇군요 ! 필수적인 부분은 아니니까 빼고 작업하는 것도 좋은 거 같아요~


class AppTypography {
static const header1 =
TextStyle(fontSize: 28, fontWeight: FontWeight.bold, height: 10 / 7);
Copy link
Collaborator

Choose a reason for hiding this comment

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

height 프로퍼티는 무슨 기준인가요??
피그마에서는 40이라고 되어 있는데 !

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

폰트 * height가 = 피그마 에 나온 값이라고 생각하시면 됩니다!

@FLU0RITE FLU0RITE merged commit 18c8041 into develop Apr 3, 2025
1 check passed
@FLU0RITE FLU0RITE deleted the chore/design_system branch May 10, 2025 02:20
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