Browse cell 일부 구현 #15
Merged
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
Jeonhui
reviewed
Nov 16, 2022
| guard let windowScene = (scene as? UIWindowScene) else { return } | ||
| let cont = BrowseViewController.create(with: BrowseViewModel()) | ||
| self.window = UIWindow(windowScene: windowScene) | ||
| self.window?.rootViewController = cont |
There was a problem hiding this comment.
이 부분은 임시로 넣어놓은 부분일거라 생각이 되는데 나중에 코디네이터 구성이 완료되면 변경하면 될 것 같습니다!
| Quest(title: "잠자기", startDay: Date(), endDay: Date(), repeat: 1, currentCount: 1, totalCount: 1) | ||
| ] | ||
|
|
||
| let data: Observable<[(User, [Quest])]> |
There was a problem hiding this comment.
typealias BrowseItem = (user:User, quests: [Quest]) 이런 식으로 각각 명칭을 지정해주면 사용할 때 조금 더 명확하게 사용할 수 있을 것 같습니다!
|
|
||
| static let reuseIdentifier = "QuestCell" | ||
| /// dequeuResusable을 위한 아이덴티파이어입니다. | ||
| static let reuseIdentifier = "UserInfoCell" |
There was a problem hiding this comment.
이건 리팩터 브랜치만들고 바로 수정할게요 :)
|
|
||
| final class QuestCell: UITableViewCell { | ||
| /// dequeuResusable을 위한 아이덴티파이어입니다. | ||
| static let reuseIdentifier = "UserInfoCell" |
| 인자로 받은 Entity Quest타입을 통해 그 정보를 기반으로 cell에 아이템을 넣습니다. | ||
|
|
||
| 애니메이션 효과가 필요없다고 판단하여, `setProgress(_:animated)`에서 두번째 인자를 | ||
| `false`로 설정하였습니다. |
There was a problem hiding this comment.
QuestCell이 home view에서 사용 됐을 경우, 사용자가 터치했을 때에는 애니메이션이 있으면 좋을 것같다는 생각이 드는데 진웅님은 어떻게 생각하시나요?:)
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.
📕 Issue Number
Close #11 #10
📙 작업 내역
Browse Cell에 헤더를 이미지와 해당 유저이름이 나타나게 변경해야하는데, PR이 너무 커질 것 같아서 분할해서 올립니다. :)
QuestCell을 구현하였습니다. QuestCell 재구성 #10
📘 작업 유형
📋 체크리스트
📝 PR 특이 사항
intrinsic content size를 사용하는 방법을 제안을 해주셔서 찾아보았으나, 아래의 코드의 숫자들이 그곳에 이동하는 형태가 될 것 같아서 코드작성을 하지 않았습니다. 😢