Skip to content

[Fix] #534 - 3.0.0 배포 버전에서의 사소한 버그들 수정 #535

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

Merged
merged 10 commits into from
Apr 3, 2025

Conversation

dlwogus0128
Copy link
Contributor

@dlwogus0128 dlwogus0128 commented Apr 3, 2025

🌴 PR 요약

3.0.0 배포 버전에서의 사소한 버그들을 수정했습니다.

🌱 작업한 브랜치

🌱 PR Point

[솝트로그 UI]

  • 솝트로그에서 툴팁이랑 솝마디 간격이 너무 붙어보여서 피그마대로 1px만.. 띄웠어요 😅
  • 솝트로그 툴팁 눌렀을 때 dim 배경 누르면 dismiss 될 수 있도록 구현했어요.
  • 솝트로그 툴팁 팝업에서의 글자 간격이 좁아보여서 수정했어요.

[홈화면 UI]

  • 홈화면에서 플그랑 프로젝트 로고 잘못 되어있어서, 피그마대로 수정했어요.
  • 그동안 대시보드에서 사용자 이름에 볼드체가 적용이 안 되어있었는데, htmlToString 한 값을 이후 lineSpacing을 세팅하면서 덮어씌워버리며 생긴 문제였어요. 그래서 UILabel에 attributedText가 적용되어 있을 경우, 해당 attributedText를 불러와 lineSpacing 속성을 추가할 수 있도록 함수를 수정했어요.

[스와이프 시 뒤로 가기]

  • 그동안 시스템의 네비바를 가리면서, 스와이프로 뒤로 가는 인터랙션이 꺼지는 문제가 있었는데 해당 부분들 전면 수정했습니다.

[푸시알림 터치 시 딥링크 화면 전환 안 됨]

  • NotificationPayload에 프로퍼티를 추가하면서, 객체가 정상적으로 생성되지 않아 생긴 문제였습니다. 문제되는 부분들 (해당 기능 구현을 위한 피처가 종료되었다고 전해들어) 삭제했습니다.

📌 참고 사항

3.0.1로 업데이트할게요!

📸 스크린샷

기능 스크린샷
수정 이후
Simulator.Screen.Recording.-.iPhone.16.Pro.Max.-.2025-04-03.at.20.51.16.mp4

📮 관련 이슈

@dlwogus0128 dlwogus0128 added Fix 문제 해결, 코드 수정 재현✦ labels Apr 3, 2025
@dlwogus0128 dlwogus0128 self-assigned this Apr 3, 2025
Copy link

height bot commented Apr 3, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Contributor

@yungu0010 yungu0010 left a comment

Choose a reason for hiding this comment

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

꼼꼼한 작업 고생하셨어요!! 🍓

Comment on lines +148 to +156
/// dimming 뒷배경을 눌렀을 때, dismiss 이벤트를 전달합니다.
override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
guard let touch = touches.first else { return }
let location = touch.location(in: self.view)
// toolTipView 프레임의 밖일 경우에만 dismiss
if !toolTipView.frame.contains(location) {
self.dimmingBackgroundTap.send()
}
super.touchesBegan(touches, with: event)
Copy link
Contributor

Choose a reason for hiding this comment

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

우와아ㅏㅇ 감사합니다 너무 좋네요👍👍 이 기능은 명세서에 없어서 따로 구현하지 않았는데, 안드로이드 측에 여쭤보고 맞춰가면 더 좋을 것 같습니다 !!

Comment on lines +117 to +121
private func setDelegate() {
self.webView.scrollView.delegate = self
self.webView.navigationDelegate = self
self.webView.uiDelegate = self
}
Copy link
Contributor

Choose a reason for hiding this comment

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

함수 분리 너무 좋네요🚀🚀

Comment on lines -15 to -17
public let title: String
public let content: String
public let sendAt: String
Copy link
Contributor

Choose a reason for hiding this comment

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

깜짝이슈ㅎㅎ,,👀

@dlwogus0128 dlwogus0128 merged commit 7b62e50 into develop Apr 3, 2025
@dlwogus0128 dlwogus0128 deleted the fix/#534-bugs branch April 3, 2025 13:03
@wsd9489
Copy link

wsd9489 commented Apr 6, 2025

영차 영차 ~ 영치기 영 차~ 파이팅입니다~~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 문제 해결, 코드 수정 size/L 재현✦
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 3.0.0 배포 버전에서의 작은 버그들 수정
3 participants