Feat/1.0.6 대비 디자인 시스템 변경사항 반영

[ShapeKim98]

#️⃣연관된 이슈

#157

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

• PokitButton 패딩, 사이즈 수정
• PokitLinkPopup 변경사항 반영
• PokitLinkPreview 스켈레톤 UI 추가
• PokitList 변경사항 반영
• PokitBadge 변경사항 반영
• PokitBookmark 컴포넌트 추가
• BaseContentItem 속성 변경
• PokitLinkCard 변경사항 반영
• 새로운 아이콘 추가

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PokitBadge 사용법이 바뀌었습니다.

• 뱃지에 아이콘만 있는 뱃지가 추가 되면서 기존의 방식이 확장성에 유연하지 못하다는 판단이 들어 변경하였습니다. 참고해주세여

#Preview {
    PokitBadge(state: .unRead)
    
    PokitBadge(state: .default("포킷명"))
    
    PokitBadge(state: .unCategorized)
    
    PokitBadge(state: .memo)
    
    PokitBadge(state: .member)
}

BaseContentItem의 속성값을 변경하였습니다.

• 변경된 PokitLinkCard는 링크의 메모와 즐겨찾기 여부를 요구하여, 메모와 즐겨찾기 여부 속성을 추가하였습니다.
• 리마인드 피처쪽 API들은 메모와 즐겨찾기 여부를 던저주지 않지만 전부 BaseContentItem으로 통일 되어 있기 때문에 메모와, 즐겨찾기 여부를 Optional로 지정하였습니다.

public protocol PokitLinkCardItem {
    var title: String { get }
    var memo: String? { get }
    var thumbNail: String { get }
    var createdAt: String { get }
    var categoryName: String { get }
    var isRead: Bool? { get }
    var data: String { get }
    var domain: String { get }
    var isFavorite: Bool? { get }
}

PokitLinkCard 사용법이 바뀌었습니다.

• 이번 변경사항에 top, middle, bottom이 새로 생기면서 기존에 구분선을 divider modifier로 구현하는 방식을 없애고, top, middle, bottom으로 구분하였습니다.

PokitLinkCard(
    link: store.content,
    state: isFirst
    ? .top
    : isLast ? .bottom : .middle,
    type: type,
    action: { send(.컨텐츠_항목_눌렀을때) },
    kebabAction: { send(.컨텐츠_항목_케밥_버튼_눌렀을때) },
    fetchMetaData: { send(.메타데이터_조회) }
)

이 pr 이후 기능 업데이트 작업 들어갈 예정입니다

[stealmh]

고생많으셨읍니다 찬찬히 읽어볼게요!!

[stealmh]

아 뭔가 좀 끌리지 않는 방식인데 해야할것도 많고 나중에 같이 생각해보시죠.
필수구현해야 하는 값을 nil을 넣는 것이 맞는것일까... 어거지로 뷰에 엮으려고 만든 프로토콜느낌이 씨게 나서요

제가 생각한건 protocol extension인데 작성한 코드를 보면 또 if let에 의해 화면에 보여져야 할 부분들이 있어서 기본값 부여가 이건 의미가 없을 것 같고 PokitLinkCardItem을 채택한 새로운 프로토콜을 만드는 것이 방법일 것 같긴 합니다 (근데 너무 귀찮을지도 모름)

...일단 인지하고 있겠습니다

[ShapeKim98]

아 뭔가 좀 끌리지 않는 방식인데 해야할것도 많고 나중에 같이 생각해보시죠. 필수구현해야 하는 값을 nil을 넣는 것이 맞는것일까... 어거지로 뷰에 엮으려고 만든 프로토콜느낌이 씨게 나서요

제가 생각한건 protocol extension인데 작성한 코드를 보면 또 if let에 의해 화면에 보여져야 할 부분들이 있어서 기본값 부여가 이건 의미가 없을 것 같고 PokitLinkCardItem을 채택한 새로운 프로토콜을 만드는 것이 방법일 것 같긴 합니다 (근데 너무 귀찮을지도 모름)

...일단 인지하고 있겠습니다

저도 조금 고민을 하긴 했는데.. 옵셔널로 넣은건 리마인드 피처 대응이고, 어차피 리마인드 피처는 없어질 피처라.. 일단 이렇게 해놓았습니다
나중에 링크 추천 기능으로 갈아엎을때 한번 더 얘기 해보죠!

[stealmh]

제가보기엔 좋아보입니다 소소한 잡담거리 놔두고 갑니다