Skip to content

[Fix] #545 - 솝탬프 다크모드 대응 QA 반영 #546

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 8 commits into from
Apr 10, 2025

Conversation

dlwogus0128
Copy link
Contributor

🌴 PR 요약

솝탬프 다크모드 대응에 대한 QA를 반영했습니다.

🌱 작업한 브랜치

🌱 PR Point

QA 받았던 부분들에 대해 수정사항을 반영했습니다.

  • 디테일뷰 색상, 간격 맞지 않는 부분을 수정했습니다.
  • 디테일뷰 도장 찍을 때 dim 처리가 누락되어 있어 추가했습니다.
  • 미션 디테일뷰에서의 자연스러운 줄바꿈을 위해서 (피그마 기준) 텍스트 count 24 이상일 경우, 문장의 middle에서 줄바꿈을 할 수 있도록 했습니다.
  • 미션 리스트 완료일 때 랭킹 버튼 눌리지 않는 부분에 대해서는, 지금 상황에서는 문제가 발견되지 않아서 보류하겠습니다.
  • 비회원일 때 안내 문구가 줄바꿈이 되지 않았던 부분은, SetLineSpacing 함수의 기존 로직이 잘못 짜여 있어서 였습니다.
    • 원래의 의도는 기존 attributedText가 있을 경우 lineSpacing를 덮어쓰는 것이 아닌 addAttribute하기 위한 것이었는데, 의도대로 동작하지 않아 내부 분기를 아래와 같이 함수로 분리했습니다.
/// 행간 조정 메서드
func setLineSpacing(lineSpacing: CGFloat) {
    if let text = self.text {
        let attributedStr = NSMutableAttributedString(string: text)
        let style = NSMutableParagraphStyle()
        style.lineSpacing = lineSpacing
        attributedStr.addAttribute(NSAttributedString.Key.paragraphStyle, value: style, range: NSMakeRange(0, attributedStr.length))
        self.attributedText = attributedStr
    }
}

/// 행간 조정 메서드: 기존 attributedText에 속성 추가
func modifyLineSpacing(lineSpacing: CGFloat) {
    if let attributedText = self.attributedText {
        let mutableAttributedString = NSMutableAttributedString(attributedString: attributedText)
        
        mutableAttributedString.enumerateAttribute(.paragraphStyle,
                                                   in: NSRange(location: 0, length: mutableAttributedString.length),
                                                   options: []) { value, range, _ in
            let paragraphStyle = (value as? NSMutableParagraphStyle) ?? NSMutableParagraphStyle()
            paragraphStyle.lineSpacing = lineSpacing
            mutableAttributedString.addAttribute(.paragraphStyle, value: paragraphStyle, range: range)
        }
        
        self.attributedText = mutableAttributedString
    }
}
  • 사진을 업로드 할 때 로딩 인디케이터가 없어, 어색하다는 피드백을 반영했습니다.

📌 참고 사항

📸 스크린샷

기능 스크린샷
미션 한 줄
미션 두 줄

📮 관련 이슈

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

height bot commented Apr 10, 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.

@dlwogus0128
Copy link
Contributor Author

배포 때문에 먼저 머지하겠습니다! 이후에 리뷰 달아주시면 반영할게요

@dlwogus0128 dlwogus0128 merged commit 48af214 into develop Apr 10, 2025
@dlwogus0128 dlwogus0128 deleted the fix/#545-soptamp-qa branch April 10, 2025 18:38
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.

꼼꼼한 QA 대응 고생하셨습니다~!!

@@ -170,7 +170,7 @@ extension MissionDateView {
}

private func getInitializedToolBar() -> UIToolbar {
let toolBar = UIToolbar(frame: CGRect(x: 0, y: 0, width: self.frame.width, height: Metric.toolBarHeight))
let toolBar = UIToolbar(frame: CGRect(x: 0, y: 0, width: Int(self.frame.width), height: Metric.toolBarHeight))
Copy link
Contributor

Choose a reason for hiding this comment

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

Int로 타입 변경을 하신 이유가 궁금해요

}

/// 행간 조정 메서드: 기존 attributedText에 속성 추가
func modifyLineSpacing(lineSpacing: CGFloat) {
Copy link
Contributor

Choose a reason for hiding this comment

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

비회원 뷰 원인이 이거 때문이었군요!!ㅎ.ㅎ

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

Successfully merging this pull request may close these issues.

[Fix] 솝탬프 다크모드 대응 QA 반영
2 participants