Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new Tooltip component to the design system, comprising TooltipView, TooltipOverlayView, and a TooltipFactory for management. It also refactors CharacterInputView and ToLoginView to utilize enums for better type safety and updates ToastFactory to an enum. The review feedback highlights opportunities to reduce code duplication regarding window access and hardcoded layout constants, optimize layer management in layoutSubviews, remove the unused TooltipArrowView file, and enhance the user experience by adding a fade-out animation during tooltip dismissal.
| ) | ||
|
|
||
| /// arrow는 툴팁 내부에서 고정 위치 | ||
| let arrowInset: CGFloat = 28 |
There was a problem hiding this comment.
이 arrowInset 값(28)은 TooltipFactory.swift의 61라인에서도 동일하게 사용되고 있습니다. 두 값이 일치하지 않을 경우 화살표의 위치와 툴팁의 배치가 어긋나는 버그가 발생할 수 있습니다. Constants 열거형에 공통 상수로 정의하여 두 파일에서 공유하도록 수정하는 것이 안전합니다.
| layer.sublayers?.removeAll(where: { $0 is CAShapeLayer }) | ||
| layer.insertSublayer(shapeLayer, at: 0) |
There was a problem hiding this comment.
layoutSubviews는 뷰의 레이아웃이 결정될 때마다 반복적으로 호출됩니다. 호출될 때마다 기존 레이어를 찾아 제거하고 새로운 CAShapeLayer를 생성하여 삽입하는 방식은 메모리 및 CPU 자원을 낭비하게 됩니다. 클래스 프로퍼티로 CAShapeLayer를 하나 유지하고, drawBubble에서는 해당 레이어의 path만 업데이트하도록 개선해 주세요.
| import UIKit | ||
|
|
||
| import SnapKit | ||
|
|
||
| final class TooltipArrowView: UIView { | ||
|
|
||
| override class var layerClass: AnyClass { | ||
| CAShapeLayer.self | ||
| } | ||
|
|
||
| override func layoutSubviews() { | ||
| super.layoutSubviews() | ||
|
|
||
| let path = UIBezierPath() | ||
| path.move(to: CGPoint(x: bounds.midX, y: bounds.maxY)) | ||
| path.addLine(to: CGPoint(x: bounds.minX, y: bounds.minY)) | ||
| path.addLine(to: CGPoint(x: bounds.maxX, y: bounds.minY)) | ||
| path.close() | ||
|
|
||
| let shape = layer as! CAShapeLayer | ||
| shape.path = path.cgPath | ||
| shape.fillColor = UIColor.whiteMLS.cgColor | ||
| } | ||
| } |
There was a problem hiding this comment.
TooltipView 내부에서 UIBezierPath를 사용하여 직접 화살표를 그리고 있으므로, 이 TooltipArrowView 클래스는 현재 프로젝트 내에서 사용되지 않는 것으로 보입니다. 불필요한 파일은 제거하여 프로젝트 구조를 깔끔하게 유지하는 것이 좋습니다.
| static var window: UIWindow? { | ||
| UIApplication.shared | ||
| .connectedScenes | ||
| .flatMap { ($0 as? UIWindowScene)?.windows ?? [] } | ||
| .first { $0.isKeyWindow } | ||
| } |
There was a problem hiding this comment.
최상단 window를 찾는 로직이 ToastFactory.swift와 완전히 중복되어 구현되어 있습니다. 향후 윈도우 탐색 로직이 변경될 경우 두 곳을 모두 수정해야 하는 번거로움이 있으므로, UIWindow extension 등으로 분리하여 공통으로 사용하는 것이 좋습니다.
| static func dismiss() { | ||
| currentTooltip?.removeFromSuperview() | ||
| currentTooltip = nil | ||
|
|
||
| overlayView?.removeFromSuperview() | ||
| overlayView = nil | ||
| } |
There was a problem hiding this comment.
show 메서드에서는 툴팁이 나타날 때 페이드 인 애니메이션이 적용되어 있지만, dismiss 시에는 즉시 제거되어 사용자 경험이 다소 부자연스러울 수 있습니다. 제거 시에도 페이드 아웃 애니메이션을 적용한 뒤 removeFromSuperview를 호출하는 것을 고려해 보세요.
1 participant
📌 이슈
✅ 작업 사항
로그인 유도 UI를 진입 경로에 따라 분기 처리할 수 있도록 수정
레벨 및 직업 입력 UI를 진입 경로에 따라 분기 처리할 수 있도록 수정
도감 리스트 셀에 랭킹 태그를 추가하기 위하여 CardList 및 TagChip 컴포넌트 수정
Tooltip 표시를 위한 컴포넌트 및 Factory 구현
+추가 : 기존 class 타입이었던 ToastFactory를 enum으로 변경하여 인스턴스 생성 방지 및 안정성 개선