알립니다 구현 #46
Merged
알립니다 구현 #46
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Closed
3 tasks
lalaurrel
approved these changes
Nov 27, 2024
| @@ -22,7 +22,7 @@ class SceneDelegate: UIResponder, UIWindowSceneDelegate { | |||
| window = UIWindow(windowScene: windowScene) | |||
|
|
|||
| let rootController = TabBarNavigationController( | |||
| rootViewController: TabBarController() | |||
| rootViewController: TicketDetailController() | |||
There was a problem hiding this comment.
혹쉬 이거 탭바로 다시 안바꿔두 돼?! 🤓
There was a problem hiding this comment.
다시 바꿔야 해 ㅋㅋㅋ 다음 타자가 바꿔줄거라고 믿고있어...
SungMinCho-Kor
requested changes
Nov 28, 2024
| @@ -1,3 +1,322 @@ | |||
| //// | |||
Comment on lines
20
to
26
| static var mockData: [InfoConfiguration] = [ | ||
| .init(title: "알립니다", details: "※ 본 공연은 네이버 쿠폰이 적용되지 않습니다.\n※ 본 공연은 YES24공연에서 진행하는 할인쿠폰이벤트 대상에서 제외됩니다.\n※ 매수제한: 공연별 1인 4매", isExpanded: false), | ||
| .init(title: "상세정보", details: ""), | ||
| .init(title: "할인정보", details: ""), | ||
| .init(title: "기대평", details: ""), | ||
| .init(title: "공연 이미지(1)", details: "") | ||
| ] |
There was a problem hiding this comment.
여기도 Control m 해주시면 좋을 것 같아요!
| import SnapKit | ||
| import Then | ||
|
|
||
| class DetailTableViewCell: UITableViewCell { |
| fatalError("init(coder:) has not been implemented") | ||
| } | ||
|
|
||
| static let identifier: String = "DetailTableViewCell" |
There was a problem hiding this comment.
UITableViewCell의 identifier도 UICollectionViewCell처럼 extension으로 구현하면 어떨까요?
Comment on lines
13
to
14
| final class ExpandedView: UIView { | ||
| private let firstLabel = UILabel().then { |
Comment on lines
88
to
90
| [ ticketDetailView, | ||
| buttonBackgroundView] | ||
| .forEach(view.addSubview) |
There was a problem hiding this comment.
줄내림과 띄어쓰기 컨벤션 확인해주세요!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎫 What is the PR?
알립니다 TicketDetailView의 전반적인 작업을 완료했습니다!
🎫 Changes
🎫 Thoughts
Expanded를 어떻게 하면 구현할 수 있을 지 공부해볼 수 있는 좋은 시간이었습니다.
TableView에 대해서 한 층 더 깊이 알아볼 수 있었던 시간..
Expanded에 더불어 재사용 셀들과 다른 하나의 셀의 화살표 동작, 펼쳐졌을 때 섹션에 점선 뷰가 나타나야 함... 등등 까다로운 부분이 많았습니다.
이를 구현하기 위해서는 테이블 뷰를 구현하기 위해 써주는 각각의 메서드들이 어떤 역할을 하는지 잘 알아야 했고, 그에 따라 조건을 걸어주어 맞는 동작을 하게 하는 것이 무엇보다도 중요했습니다. 그리고 각각의 화면에 그려지는 View, layer의 차이도 조금 더 명확히 알 수 있었던 시간이었습니다.
그러니, 저의 코드에서 TableView에서 조건을 걸어준 부분을 조금 유심하게 봐 주셨으면 좋겠습니다.
그리고 Footer를 TableViewHeaderFooterView로 붙여주기 위한 snp도 공부할 수 있었습니다.
🎫 Screenshot
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2024-11-27.at.09.47.43.mp4
🎫 To Reviewers
TableView 구현 조건 위주로 봐주세요!
🎫 Related Issues
진짜 까다로웠다... 드디어 하나 끝냈따...