[Fix] #410 - SE기기대응, 솝마디 토스트 메세지 추가 #412
Conversation
|
.../Projects/Features/DailySoptuneFeature/Sources/DailySoptuneScene/VC/DailySoptuneMainVC.swift
Outdated
Show resolved
Hide resolved
제가 기기 대응 많이 안해봐서 잘 모르긴 하는데 ...이미지나 카드 같은 것들 기기 배율 정할 때.. adjusted로 width값 정하고, 그 비율에 맞춰서 height 정하는 건 어떠실까요? 예를 들어서 todayFortuneImage.snp.makeConstraints { make in
make.top.equalTo(recieveFortune.snp.bottom).offset(9.adjustedH)
make.leading.trailing.equalToSuperview().inset(53.adjusted)
make.height.equalTo(208.adjustedH)
}위처럼 되어있던 것을, 피그마에 나와 있는 비율 값을 계산해서 (270 / 307 = 0.87) todayFortuneImage.snp.makeConstraints { make in
make.top.equalTo(recieveFortune.snp.bottom).offset(9.adjustedH)
make.leading.trailing.equalToSuperview().inset(53.adjusted)
make.height.equalTo(width * 0.87)
}이런 식으로요! 저는 아래와 같이 private enum Metric {
static let soptuneLogoWidth = 133.48.adjusted
static let soptuneLogoRatio = 102.71 / 133.48
}요롷게 비율을 처리하고 있긴 한데...! soptuneLogoImage.snp.makeConstraints { make in
make.top.equalToSuperview().inset(32.adjustedH)
make.width.equalTo(Metric.soptuneLogoWidth)
make.height.equalTo(Metric.soptuneLogoWidth * Metric.soptuneLogoRatio)
make.centerX.equalToSuperview()
}별로다~ 싶으면 쓰루해주셔도 될 것 같습니다 ㅎ.ㅎ |
|
오 이게 훨씬 좋은 방법 같아요 ! 안그래도 adjusted와 adjustedH를 혼합해서 사용하니까 비율이 안 맞고 너무 못생겨져서 고민이 되었었는데,, 높이는 재현님이 말씀해주신 방법대로 적용해보겠습니다 ! 좋은 의견 감사해요🫶 |
yungu0010
changed the title
| @@ -375,6 +375,7 @@ public struct I18N { | |||
| public static let recieveTodayFortune = "오늘의 운세가 도착했어요!" | |||
| public static let checkTodayFortune = "오늘의 운세 확인하기" | |||
| public static let goHome = "홈으로 돌아가기" | |||
| public static let dateErrorToastMessage = "앗, 오늘의 솝마디만 볼 수 있어요." | |||
| } | |||
| if let deepLink = owner.notification?.deepLink, | ||
| let date = owner.notification?.createdAt { | ||
| if !owner.isToday(date.toDate()) && deepLink.hasSuffix("fortune") { | ||
| ToastUtils.showMDSToast(type: .alert, text: I18N.DailySoptune.dateErrorToastMessage) |
There was a problem hiding this comment.
여기까지 guard let 으로 간결하게 나타낼 수도 있울 것 같은데 어떠신가요??
guard let deepLink = owner.notification?.deepLink,
let date = owner.notification?.createdAt,
!owner.isToday(date.toDate()),
deepLink.hasSuffix("fortune")
else { return true }요롷게...!
There was a problem hiding this comment.
헉 넘 좋아요! 바로 반영하겠습니다🫶
| } | ||
| .store(in: cancelBag) | ||
| output.pokeResponse.send(pokeUserModel) | ||
| }.store(in: cancelBag) |
There was a problem hiding this comment.
고맙숩뉘다아아아아아아ㅏ....ㅜㅜㅜㅜ , ㅠ,
🌴 PR 요약
🌱 작업한 브랜치
📌 참고 사항
adjusted 또는 adjustedH를 함께 사용하고 있는데, 컴포넌트의 비율을 기존과 같이 유지하려면 둘 중 하나만 써야합니다.! 하지만 그렇게 할 경우 SE 화면 속에서 스크롤뷰 없이 기기대응이 불가합니다.우선 adjusted와 adjustedH를 혼합하여 사용했는데, 화면에서 차지하는 비율을 기준으로 레이아웃을 잡다보니 다른 기기들과 비율이 맞지 않습니다. . -> 솝마디 메인뷰와 솝마디 카드뷰도 스크롤뷰로 바꾸어야 할까요?버튼은 알림 상세와 통일성을 가지기 위해 기기대응을 적용하지 않았습니다.
디자이너 분들 의견에 따라 MainVC는 스크롤뷰 + 플로팅 버튼, CardVC는 스크롤뷰 적용하였습니다.
마찬가지로 디자이너 분들 의견에 맞게 dismiss 버튼은 스크롤뷰에 추가하지 않았습니다!
UseCase에서 처리하려고 했으나, 버튼을 눌렀을 때 토스트 메세지를 띄워야한다고 해서 ViewModel에서 작업하였습니다.
📸 스크린샷
📮 관련 이슈