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

[Fix] #17 - CustomNavigationBar 수정, typoStyle 추가 #18

Merged

Conversation

yangsubinn
Copy link
Member

🌴 PR 요약

  • CustomNavigationBar 수정
  • typoStyle 추가
  • 에셋 추가

🌱 작업한 브랜치

🌱 PR Point

1️⃣ TypoStyle

TypoStyle은 이미 들어가있던 타이포 스타일 외에 두개만 추가했습니다(caption1D, caption2D)
스크린샷 2022-11-27 오후 10 42 04
피그마의 디자인 시스템 내에서는 타이포 이름에 _(언더바)하고 weight랑 size가 추가되어있는데,
저희가 사용할 때는 _(언더바) 뒷부분은 무시하고 보시면 될 것 같습니다

ex) h1_b20 -> h1
cap1_r14 -> caption1
cap1_d_m14 -> caption1D

2️⃣ CustomNavigationBar

  1. naviType
@frozen
public enum naviType {
    case title /// 좌측 타이틀 + 우측 버튼 (미션 리스트 뷰)
    case titleWithLeftButton /// 좌측 뒤로가기 버튼 + 좌측 타이틀 (랭킹, 글 작성 등)
}
  1. rightButtonType
@frozen
public enum rightButtonType {
    case none // rightButton 없는 경우
    case addRecord // 글 열람 뷰 (수정 버튼)
    case delete // 글 수정 뷰 (삭제 버튼)
}

setRightButton(_ type: rightButtonType)에서 설정해서 사용할 수 있습니다

사용예시

스크린샷 2022-11-28 오전 12 12 19

네비바 종류가 크게 달라지지 않을 것 같아서 이 세개만 있다는 가정하에 분류했습니다
// 첫번째 네비바
lazy var naviBar = CustomNavigationBar(self, type: .title).setTitle("미완료 미션")

// 두번째 네비바
lazy var naviBar = CustomNavigationBar(self, type: .titleWithLeftButton).setTitle("미션").setRightButtonType(.none)

// 세번째 네비바
lazy var naviBar = CustomNavigationBar(self, type: .titleWithLeftButton).setTitle("미션").setRightButtonType(.addRecord)

📌 참고 사항

📸 스크린샷

기능 스크린샷
기능이름 스크린샷 첨부

📮 관련 이슈

@yangsubinn yangsubinn added Fix 문제 해결, 코드 수정 양수🌷 labels Nov 27, 2022
@yangsubinn yangsubinn self-assigned this Nov 27, 2022
Copy link
Member

@devxsby devxsby left a comment

Choose a reason for hiding this comment

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

감사합니당 🫶🏻
저희 근데 라이트모드랑 세로모드고정도 되어있나여?
안되어있으면 요기 브랜치에서 같이 작업하는것은 어떠할까여,,??!

Copy link
Contributor

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

오아 너무 편리하네여...감사합니다!!

Merge branch 'develop' of https://github.com/sopt-makers/SOPT-Stamp-iOS into fix/sopt-makers#17-CustomNavigationBar
Copy link
Member

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

감사합니다👍

@yangsubinn
Copy link
Member Author

라이트모드랑 세로모드까지 고정했습니다 ! @L-j-h-c @lsj8706 @devxsby

@yangsubinn yangsubinn merged commit c5d140e into sopt-makers:develop Nov 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 문제 해결, 코드 수정 size/L 양수🌷
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] CustomNavigationBar 수정, TypoStyle 수정
4 participants