Skip to content

Conversation

@hsw1920
Copy link
Collaborator

@hsw1920 hsw1920 commented Nov 11, 2024

🤔 배경

커스텀 프레임, 스티커 버튼 공통 컴포넌트가 필요했습니다.

📃 작업 내역

  • SnapKit 라이브러리를 DesignSystem 모듈에 의존성을 추가했습니다.
  • 필요한 Icon을 위해 Asset을 추가했습니다.
  • Asset의 Icon과 Color를 폴더링했습니다.
  • 공통 컴포넌트 class를 구현했습니다.

✅ 리뷰 노트

SnapKit 라이브러리 의존성을 DesignSystem에 추가 했을 때 발생하는 Warning

"Module 'SnapKit' was not compiled with library evolution support; using it means binary compatibility for 'DesignSystem' can't be guaranteed"

image

  • Feature는 DesignSystem을 모두 의존하는 상황이었기 때문에 SnapKit을 DesignSystem 모듈에 의존성을 추가하였습니다.
  • 해당 워닝의 의미를 잘 모르겠습니다..

🎨 스크린샷

  • 아래 스크린샷은 EditPhotoRoomFeatureDemo에서 인스턴스를 추가하고 레이아웃만 확인하였습니다.
iPhone 16

🚀 테스트 방법

  • 실행할 수 있는 모듈에서 DesignSystem 모듈을 의존하여 적절히 인스턴스를 추가하고 레이아웃을 잡아서 확인해주세요
    let button = PTGGrayButton(type: .sticker)
    
    public override func addViews() {
        view.addSubview(button)
        view.addSubview(button2)
    }
    
    public override func setupConstraints() {
        button.snp.makeConstraints {
            $0.height.equalTo(52)
            $0.leading.equalToSuperview()
            $0.width.equalTo(150)
            $0.bottom.equalTo(view.safeAreaLayoutGuide)
        }
    }

hsw1920 and others added 3 commits November 11, 2024 16:56
- SnapKit 추가
- 최소 지원 16.0으로 변경
- Only iOS

Co-Authored-By: YeongHoon Song <[email protected]>
- PTGIcon 추가
- ColorSet, IconSet 폴더링

Co-Authored-By: YeongHoon Song <[email protected]>
- enum type을 통해 PTGGrayButton의 "프레임" 또는 "스티커" 타입의 버튼을 생성시 설정할 수 있습니다.
- UIImage+ extension을 추가하여 외부 번들에서 DesignSystem의 IconAsset을 로드할 수 있습니다.

Co-Authored-By: YeongHoon Song <[email protected]>
@hsw1920 hsw1920 added the ✨ feat 새로운 기능 추가 label Nov 11, 2024
@hsw1920 hsw1920 linked an issue Nov 11, 2024 that may be closed by this pull request
1 task
@hsw1920 hsw1920 linked an issue Nov 11, 2024 that may be closed by this pull request
1 task
Copy link
Member

@youn9k youn9k left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@Kiyoung-Kim-57 Kiyoung-Kim-57 left a comment

Choose a reason for hiding this comment

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

LGTM!

@hsw1920 hsw1920 merged commit 77a71af into develop Nov 11, 2024
@0Hooni 0Hooni deleted the style/#8-frame-sticker-button-UI branch November 11, 2024 08:53
@hsw1920 hsw1920 restored the style/#8-frame-sticker-button-UI branch November 11, 2024 08:57
@hsw1920 hsw1920 deleted the style/#8-frame-sticker-button-UI branch November 11, 2024 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

프레임, 스티커 공통 UI 컴포넌트를 구현한다

5 participants