[WEAV-136] ToolTip AttibutedString 적용 #42
Conversation
Walkthrough이번 변경 사항은 여러 파일에서 툴팁 기능을 추가하거나 수정하는 내용을 포함하고 있습니다. Changes
Suggested labels
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (2)
🔇 Additional comments (2)Projects/Features/Home/UnitTest/WidgetUnitTest.swift (2)
#!/bin/bash
# Description: Check for content initialization patterns in widget-related files
ast-grep --pattern 'WidgetWritingIntent.*init.*content.*nil'
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (7)
Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputIntent.swift (1)
79-81: 옵셔널 처리 개선이 필요합니다
model이 weak 참조되어 있으므로, nil 체크를 추가하는 것이 좋습니다.다음과 같이 수정하는 것을 제안합니다:
func toggleToopTip() { - model?.toggleToolTip() + guard let model else { + Logger.debug("모델이 해제되었습니다") + return + } + model.toggleToolTip() }Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputModel.swift (1)
41-41: 구현이 깔끔하나 문서화가 필요합니다.툴팁 기능의 구현이 간단하고 명확합니다. 하지만 다른 개발자들의 이해를 돕기 위해 문서화를 추가하면 좋을 것 같습니다.
다음과 같이 문서 주석을 추가하는 것을 제안합니다:
@Published var targetGender: GenderType = .female + /// 나이 입력 툴팁의 표시 여부를 관리하는 상태값 @Published var isShowToolTip: Bool = false // default @Published var isLoading: Bool = false+ /// 툴팁의 표시 상태를 토글합니다. + /// 현재 표시 중이면 숨기고, 숨겨져 있으면 표시합니다. func toggleToolTip() { isShowToolTip.toggle() }Also applies to: 85-87
Projects/DesignSystem/DesignCore/Sources/Tooltip/ToolTip.swift (2)
33-43: String 초기화 로직을 더 간단하게 개선할 수 있습니다.현재 구현된 String 초기화 메서드의 nil 처리 로직을 map을 사용하여 더 간결하게 작성할 수 있습니다.
다음과 같이 개선해보세요:
init( message: String?, offset: CGFloat ) { - if let message { - self.message = .init(stringLiteral: message) - } else { - self.message = nil - } + self.message = message.map { AttributedString(stringLiteral: $0) } self.offset = offset }
48-68: 접근성 및 레이아웃 관련 개선사항이 필요합니다.
고정 너비 제거로 인한 잠재적 문제:
- 매우 긴 텍스트의 경우 화면을 벗어날 수 있습니다
- 최대 너비 제한을 고려해보세요
접근성 개선 필요:
- VoiceOver 지원을 위한 accessibilityLabel 추가
- 툴팁의 역할을 명확히 하는 accessibilityRole 설정
다음과 같이 개선해보세요:
if let message { Text(message) .padding(.horizontal, 20) .padding(.vertical, 10) .typography(.regular_12) .foregroundStyle(.white) .multilineTextAlignment(.center) + .frame(maxWidth: 300) + .accessibilityLabel("도움말: \(message)") + .accessibilityRole(.help)Projects/Model/Model/Sources/SignUp/Domain/SignUpFormDomain.swift (1)
163-168: 문자열 현지화 적용을 고려해보세요.현재 성별 표시 문자열이 하드코딩되어 있습니다. 앱의 국제화를 위해 Localizable.strings 파일을 사용한 현지화 처리를 권장드립니다.
다음과 같이 변경하는 것을 고려해보세요:
public var name: String { switch self { - case .male: "남성" - case .female: "여성" + case .male: NSLocalizedString("gender.male", comment: "Male gender") + case .female: NSLocalizedString("gender.female", comment: "Female gender") } }Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputView.swift (2)
40-58: AttributedString 구현을 개선하세요.텍스트 범위를 찾는 현재 방식은 텍스트가 변경되면 실패할 수 있습니다. 더 안정적인 구현을 위해 다음과 같이 수정하는 것이 좋습니다.
private var tooltipMessage: AttributedString { - let text = """ - 2024년 기준으로 2004년생(만 20살)부터 - 1989년생(만 35살)까지 가입할 수 있어요 - """ - - var attributedString = AttributedString(text) - let boldRanges = [ - text.range(of: "2004년생(만 20살)부터"), - text.range(of: "1989년생(만 35살)") - ].compactMap { $0 } - - boldRanges.forEach { range in - let attributedRange = attributedString.range(of: text[range])! - attributedString[attributedRange].inlinePresentationIntent = .stronglyEmphasized - } - - return attributedString + var young = AttributedString("2004년생(만 20살)부터") + young.inlinePresentationIntent = .stronglyEmphasized + + var old = AttributedString("1989년생(만 35살)") + old.inlinePresentationIntent = .stronglyEmphasized + + return young + AttributedString("\n") + old + AttributedString("까지 가입할 수 있어요") }
104-107: 툴팁 토글 기능이 잘 구현되었습니다.애니메이션과 조건부 렌더링이 적절히 구현되었습니다. 하지만 접근성을 위해 다음 사항을 고려해보세요:
Button(action: { withAnimation { intent.toggleToopTip() } }, label: { HStack(spacing: 4) { DesignCore.Images.iconInformation.image Text("가입 연령 확인하기") .typography(.regular_14) .foregroundStyle(DesignCore.Colors.grey200) } }) +.accessibilityLabel("가입 가능 연령 정보") +.accessibilityHint("버튼을 눌러 가입 가능한 연령대를 확인하세요")Also applies to: 119-119
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (6)
.gitignore(1 hunks)Projects/DesignSystem/DesignCore/Sources/Tooltip/ToolTip.swift(1 hunks)Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputIntent.swift(2 hunks)Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputModel.swift(4 hunks)Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputView.swift(4 hunks)Projects/Model/Model/Sources/SignUp/Domain/SignUpFormDomain.swift(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- .gitignore
🔇 Additional comments (5)
Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputModel.swift (1)
24-24: 프로토콜에 툴팁 상태 추가가 적절합니다!
상태 관리를 위한 프로토콜에 툴팁 표시 여부를 추가한 것이 적절해 보입니다.
Projects/DesignSystem/DesignCore/Sources/Tooltip/ToolTip.swift (1)
12-16: 구현이 깔끔하고 명확합니다!
String과 AttributedString을 모두 지원하도록 메서드를 분리한 것이 좋은 접근 방식입니다. SwiftUI의 관례를 잘 따르고 있습니다.
Projects/Model/Model/Sources/SignUp/Domain/SignUpFormDomain.swift (1)
156-162: 접근 제어자 변경이 적절합니다.
toDto 프로퍼티를 public으로 변경한 것은 툴팁 기능 구현을 위해 필요한 변경으로 보입니다.
Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputView.swift (2)
65-65: 성별 표시가 적절히 구현되었습니다.
상태 관리와 문자열 보간이 올바르게 구현되었습니다.
144-148: 툴팁 닫기 제스처가 적절히 구현되었습니다.
상태 확인 후 툴팁을 닫는 제스처가 올바르게 구현되었습니다.
Projects/Features/SignUp/Sources/ProfileInput/AuthProfileAge/AuthProfileAgeInputIntent.swift
Outdated
Show resolved
Hide resolved
구현사항
스크린샷(선택)
Summary by CodeRabbit
새로운 기능
name이 추가되었습니다.버그 수정
GenderType열거형의toDto속성 접근 수준이 변경되어 외부에서 접근 가능해졌습니다.문서화