Like animation #80
Merged
Like animation #80
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
- ContentView 자체를 커스텀해서 content view를 파라미터로 받아서 팝업에 표시하도록 구현 - left, right -> top, bottom으로 수정 - withSeprator 플래그를 만들어서 구분선 유뮤 구현 - bottom button(마지막 버튼)은 무조건 separator 있음
- Toast-Swift 참고 - 디폴트 값 설정
- 서버 연동 전까지는 해당 이미지로 테스트하고자 함
- user info box 팝업에서 report(느낌표) 버튼 클릭 시, 멈춤 화면 디자인 반영 - 차단 or 신고 팝업 표시 시, 멈춤 화면에서 이미지, 텍스트 히든 - 중지했을 때의 동작을 enum으로 나눠서 처리할 필요 없는 거 같아서 bool로 변경 - 차단 or 신고 시, 멈춤 화면 없애고 dim view 표시 후, none 상태로 설정
- footer와 section의 제약조건 설정
- TimeState에 none 케이스 생성 - 좋아요, 싫어요, 신고, 차단 시, 타이머, 프로그래스를 초기화하기 위함 - 13초에서 15초로 디자인 반영 - vc에서는 scroll만 처리하고 나머지는 cell에서 처리할 수 있도록 구현
ibcylon
approved these changes
May 30, 2024
|
|
||
| private func addGradientBackground() { | ||
| let gradientLayer = CAGradientLayer() | ||
| gradientLayer.frame = bounds |
There was a problem hiding this comment.
그라디언트가 보이지 않는 건 오토레이아웃이 정해지기전의 바운드 값이 들어가서 그렇습니다.
layoutSubView에서 gradientLayer frame값을 새로 넣어주시거나해야합니다.
gradientLayer class member로 빼셔서요
- borderWidth 삭제 - cgcolor로 설정
- 최근 클릭 후, 5초 딜레이 - 필요없는 코드 삭제
Minny27
commented
May 30, 2024
| @@ -193,6 +193,7 @@ final class FallingUserCollectionViewCell: TFBaseCollectionViewCell { | |||
| .disposed(by: disposeBag) | |||
|
|
|||
| output.rejectButtonAction | |||
| .throttle(.milliseconds(5000), latest: false) | |||
There was a problem hiding this comment.
유저 좋아요, 싫어요 최근 이벤트 후, 5초 딜레이 시켰습니다.
| } | ||
|
|
||
| required init?(coder: NSCoder) { | ||
| fatalError("init(coder:) has not been implemented") | ||
| override func layoutSubviews() { |
There was a problem hiding this comment.
bounds가 설정된 다음에 layer의 frame을 할당하도록 수정했습니다.
| gradientLayer.colors = [ | ||
| DSKitAsset.Color.DummyUserGradient.backgroundFirst.color, | ||
| DSKitAsset.Color.DummyUserGradient.backgroundSecond.color, | ||
| DSKitAsset.Color.DummyUserGradient.backgroundFirst.color |
There was a problem hiding this comment.
cgColor가 아니면 색상이 적용되지 않는 문제가 있었습니다..
| } | ||
|
|
||
| private func configureUI() { | ||
| layer.borderWidth = 1 |
There was a problem hiding this comment.
layer를 추가하는 것이다보니 borderWidth가 추가적으로 설정되는 문제가 있어 삭제했습니다.
|
더미 데이터 footerview 스크린샷입니다. |
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.
Motivation ⍰
Key Changes 🔑
기존에는 cell에서 처리한 좋아요, 싫어요에 대한 scroll 및 ui변화를 vc(상위 뷰)에서 처리했었는데, cell에서 처리할 ui업데이트는 cell에서 처리하고, 스크롤 또는 셀 삭제 등만 vc(상위 뷰)에서 처리할 수 있도록 개선했습니다.
현재 footer에 배경과 테두리 그레디언트를 구현은 했으나, 적용이 안 되는데 왜 그런지 모르겠습니다..
To Reviewers 🙏🏻
Linked Issue 🔗