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

[DesignKit] 파일 생성및 리소스 파일 생성 #9

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jungseungyeo
Copy link
Member

@jungseungyeo jungseungyeo commented Feb 8, 2025

Color asset 추가 및 다이나믹 framework로 변경

변경 유형

  • 버그 수정
  • 새로운 기능
  • 리팩토링
  • 문서 업데이트

변경 사항

  • DPUI 코드 추가, rx를 사용하는것처럼, UIColor.dpUI.background, Color.dpUI.background등 dpUI를 통해서 사용하여 중간 depdency를 추가하는 작업

  • 시작 코드 참조,

  • DesigneKit을 staticframework에서 dynamicframework로 변경, 리소스 파일을 참조하고, 전역적으로 사용하는 구조라 다이나믹이 필요한 것으로 판단,

관련링크 (JIRA)

  • 없음

관련 로그, 스크린샷

작업전 작업후
작업전 작업후

Color asset 추가 및 다이나믹 framework로 변경
@jungseungyeo jungseungyeo added the ✨ Feature 기능 개발 레이블입니다. label Feb 8, 2025
@jungseungyeo jungseungyeo self-assigned this Feb 8, 2025
@jungseungyeo jungseungyeo requested a review from a team as a code owner February 8, 2025 23:12
Comment on lines +12 to +19
public struct DPUI<Base> {

public let base: Base

public init(_ base: Base) {
self.base = base
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

DP는 디프만 약자인가요?? DPUI가 뭐지 하고 찾아봤습니다 네이밍 관련해서 조금 논의해보고 가는 것도 좋을 것 같아요

Copy link
Member Author

Choose a reason for hiding this comment

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

네이밍은 변경이 필요한거같애 아직 서비스 이름이 나오지 않아서 일단 쓰긴했는데 같이 논의해서 변경해도될거같애 ㅎㅎ 구조는 어떻게 생각해??

Copy link
Collaborator

Choose a reason for hiding this comment

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

구조는 좋은 것 같아 UIColor, Color 사용하는 쪽에서 접근 방법이 명확해서 헷갈릴 일이 없을 것 같아 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 레이블입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants