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

[Feature] WhiteboardList 당겨서 새로고침 구현 및 다크모드 대응 #116

Merged
merged 3 commits into from
Nov 26, 2024

Conversation

ekrud99
Copy link
Collaborator

@ekrud99 ekrud99 commented Nov 26, 2024

🌁 Background

  • 기존 서칭 로직으로의 복귀로 인해 부득이하게 당겨서 새로고침 기능을 추가했습니다.
  • 컬렉션뷰 및 컬렉션뷰 셀 다크모드 대응을 추가했습니다.

📱 Screenshot

  • 각 시뮬레이터간의 연관성으로 인해 하나의 영상으로 대체합니다.
  • 왼쪽부터 실 기기, 16 Pro, SE3, 13 mini 입니다.
2024-11-26.5.13.52.mov

👩‍💻 Contents

  • 당겨서 새로고침 구현
    • 로직은 Searching 정지, 기존 foundPeers 삭제 후 다시 Searching하는 방식입니다.
    • 삭제하지 않으면 serviceBrowser가 꺼졌을 때 사라진 피어를 찾지 못하기 때문입니다.
  • 컬렉션뷰와 컬렉션뷰 셀의 다크모드 대응

📣 Related Issue

@ekrud99 ekrud99 self-assigned this Nov 26, 2024
Copy link
Collaborator

@choijungp choijungp left a comment

Choose a reason for hiding this comment

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

어푸푸 어푸푸푸푸 어푸푸 어푸푸루부 ~~ 😇👍🏻
수고하셨습니따우니 ~

@@ -28,7 +28,7 @@ class WhiteboardCell: UICollectionViewCell {
private let titleLabel: UILabel = {
let label = UILabel()
label.font = AirplainFont.Subtitle2
label.textColor = .airplainBlack
label.textColor = .black
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 녀석은 다크모드 때문에 어쩔 수 없었떤 것이었을까요 ..................... Color Asset에 다크모드 추가하는 건 어떤지 ....

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

상단 AirPlaIN 타이틀 때문에 다크모드에 흰색 컬러를 추가해놨는데..
그렇게 하면 다크모드 전환 시 cell의 "다우니의 보드" 이 제목이 흰색으로 바뀌는 이슈가 있습니다..

Copy link
Collaborator

Choose a reason for hiding this comment

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

오키 ~~~ ㅠㅠ 유감이라고 판단 .... 넘어가시죠

Copy link
Collaborator Author

@ekrud99 ekrud99 Nov 26, 2024

Choose a reason for hiding this comment

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

다크모드 전환해도 기존 컬러 유지 이런 방법이 있네유..

@@ -253,6 +261,13 @@ public final class WhiteboardListViewController: UIViewController {
}
.store(in: &cancellables)
Copy link
Collaborator

Choose a reason for hiding this comment

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

얘 띵똥이 죽이지 않았나요 ??? whiteboardPublisher ??

Copy link
Collaborator

Choose a reason for hiding this comment

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

MPC 버그 이슈를 롤백하면서 같이 부활했나봐요 ㅋㅋㅋ큐ㅠㅠ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

삭제 완

@@ -28,7 +28,7 @@ class WhiteboardCell: UICollectionViewCell {
private let titleLabel: UILabel = {
let label = UILabel()
label.font = AirplainFont.Subtitle2
label.textColor = .airplainBlack
label.textColor = .black
Copy link
Collaborator

Choose a reason for hiding this comment

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

이렇게 되면 다크모드로 전환했을 때 검정색으로 나오지 않을까요???
.airpainBlack은 다크모드에서 흰색으로 표시되는 것으로 알고 있습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

검정색을 의도하고 수정했습니다.
cell은 다크모드를 켜도 배경이 흰색이라 다크모드 전환시에도 타이틀의 컬러는 검정색이어야하는데, airplainBlack은 다크모드 색상이 흰색입니다..

Copy link
Collaborator

Choose a reason for hiding this comment

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

아하 기본 Cell의 배경이 흰색이군요 👍👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Comment on lines +267 to +269
DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { [weak self] in
self?.refreshControl.endRefreshing()
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

실제로는 화이트보드 리프레쉬가 완료되지 않았는데, endRefreshing이 되는 경우도 있을 것 같습니다.
물론 저희 로직이 실제 리프레쉬랑은 약간 다른 부분이 있긴 하지만요!! 일단 킵 고잉~

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

킵코잉~

@ekrud99 ekrud99 merged commit ad30a11 into develop Nov 26, 2024
2 checks passed
Copy link
Member

@eemdeeks eemdeeks left a comment

Choose a reason for hiding this comment

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

고생하셨습니다ㅏ

@@ -28,7 +28,9 @@ class WhiteboardCell: UICollectionViewCell {
private let titleLabel: UILabel = {
let label = UILabel()
label.font = AirplainFont.Subtitle2
label.textColor = .airplainBlack
let originColor = UIColor(named: "AirplainBlack") ?? .black
Copy link
Member

Choose a reason for hiding this comment

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

어라 혹시 .airplainbalck 말고 사용하신 이요눈 뭘까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이 색상은 다크모드, 라이트모드 관계 없이 검정색으로 유지되어야합니다.
이유는 cell의 색상이 흰색이라 그냥 .airplainBlack 상태에서 다크모드 전환 시 글자가 안보이는 이슈가 있었습니다!

@ekrud99 ekrud99 deleted the feature/RefreshWhiteboardList branch November 26, 2024 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 화이트보드 셀 UI 구현 [Feature] 화이트보드 탐색 구현
4 participants