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

feat: 홈 scene 생성, 탭바 설정 #16

Merged
merged 8 commits into from
Nov 15, 2023
Merged

feat: 홈 scene 생성, 탭바 설정 #16

merged 8 commits into from
Nov 15, 2023

Conversation

loinsir
Copy link
Collaborator

@loinsir loinsir commented Nov 15, 2023

🧑‍🚀 PR 요약

해당 pr에서 작업한 내역을 적어주세요.

  • 홈 scene 생성
  • 탭바 UI 설정

탭바랑 홈 Scene 생성하고, 간단하게 업로드 버튼만 올려놨습니다.
혹시 추후 각 Scene을 연결하게 되면 탭바 -> UINavigationController -> Scene 순으로 연결해주어야 한다는 것 잊지마세요!

📸 ScreenShot

작동, 구현화면

Linked Issue

close #15

Comment on lines 1 to 8
{
"images" : [
{
"filename" : "upload_button.png",
"idiom" : "universal",
"scale" : "1x"
},
{
Copy link
Member

Choose a reason for hiding this comment

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

엇 버튼 에셋으로 만드신 이유가 있나요 아니면 나중에 깎으실 예정?
큰 상관은 없지만 나중에 색상이나 디자인 바뀌면 에셋 자체가 바뀌어야할 것 같아서용
floating 버튼 느낌으로 만들어놓으면, 지도뷰에서도 재사용할 수 있을 것 같기도 하고요 ㅎ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

floating으로 만들어 볼까요... 함 해보겠습니다. 아예 생각을 안했었네요

Copy link
Collaborator Author

@loinsir loinsir Nov 15, 2023

Choose a reason for hiding this comment

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

LOCircleButton으로 구현했습니다. 다음과 같이 사용하시면 됩니다.

let button = LOCircleButton(style: .add, diameter: 52) // 선언 시
button.makeRounded(to: 100) // 크기 변경 시

Comment on lines +11 to +36
class BaseViewController: UIViewController {

override func touchesEnded(_ touches: Set<UITouch>, with event: UIEvent?) {
super.touchesEnded(touches, with: event)
self.view.endEditing(true)
}

// MARK: - ViewController Life Cycle

override func viewDidLoad() {
super.viewDidLoad()
setConstraints()
setUI()
}

// MARK: - Methods

func setConstraints() {
// Set AutoLayout
}

func setUI() {
// Set UI
view.backgroundColor = UIColor.background
}
}
Copy link
Member

Choose a reason for hiding this comment

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

gooood 좋습니당..

Copy link
Collaborator

@chopmozzi chopmozzi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~~~ 고칠 부분은 없는 것 같아 Approve박겠슴다~~

Comment on lines +11 to +15
extension UIView {
func addSubviews(_ views: UIView...) {
views.forEach { addSubview($0) }
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

좋네용
넣는 김에 translates~~~머시기 false박는 부분도 있으면 좋을것 같네요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

엄... 이 메서드는 단순히 여러 메서드를 뷰에 추가시키는 단일 용도로만 사용하는게 좋지 않을까요?

Copy link
Collaborator

@chopmozzi chopmozzi left a comment

Choose a reason for hiding this comment

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

이미지셋 폴더 처리에 관해서도 한번 통일을 해야 할 것 같습니다. 저도 뭐 하나 만들어둬가지고..

Comment on lines +25 to +26
private(set) var diameter: CGFloat

Copy link
Collaborator

Choose a reason for hiding this comment

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

뭘 수정했나 했네요 좋습니다


private(set) var diameter: CGFloat

var style: Style {
Copy link
Member

Choose a reason for hiding this comment

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

init할 때를 제외하고 style을 외부에서 변경해주는 일이 혹시 있을까요? 🤔

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

혹시 몰라 좀 더 유연하게 만들어주고 싶었습니다

@loinsir loinsir linked an issue Nov 15, 2023 that may be closed by this pull request
3 tasks
}
}

private func setImage(by style: Style) {
Copy link
Member

@anyukyung anyukyung Nov 15, 2023

Choose a reason for hiding this comment

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

메소드마다 style 인자로 받는건 style에 따라 달라진다는걸 명시적으로 표현하기 위함이신거겠져 ???
다른 컴포넌트에 있는 style관련 메소드들도 이렇게 통일해도 괜찮을 것 같네요

Copy link
Collaborator Author

@loinsir loinsir Nov 15, 2023

Choose a reason for hiding this comment

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

메소드마다 style 인자로 받는건 style에 따라 달라진다는걸 명시적으로 표현하기 위함이신거겠져 ???

네 맞아요. 의도를 잘 헤아려 주셨네요 👍

Copy link
Member

@anyukyung anyukyung left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 !!

@loinsir loinsir merged commit a7c51ad into iOS/dev Nov 15, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: BaseViewController, 탭바
3 participants