Skip to content
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

[iOS] 지도 그라디언트 #348

Merged
merged 22 commits into from
Jan 12, 2024
Merged

[iOS] 지도 그라디언트 #348

merged 22 commits into from
Jan 12, 2024

Conversation

SwiftyJunnos
Copy link
Member

@SwiftyJunnos SwiftyJunnos commented Jan 11, 2024

❗ 배경

작업 배경에 대한 설명을 작성합니다.
Issue에 대한 링크를 첨부합니다.

🔧 작업 내역

작업한 내용들을 나열합니다.
간결하게 리스트 업하고, 자세한 설명은 아래 리뷰 노트에서 합니다.

  • 지도 하단 그라디언트 추가

📝 리뷰 노트

작업 내역에 대한 자세한 설명을 작성합니다.

MSGradientLayer

기존 CAGradientLayer는 colorsCGColor를 사용합니다.

때문에 UIColor를 사용할 수 있는 간단한 커스텀 클래스를 만들어줬습니다.

import UIKit

public class MSGradientLayer: CAGradientLayer {
    
    // MARK: - Properties
    
    public var gradientColors: [UIColor] = [] {
        didSet { self.updateColors() }
    }
    
    // MARK: - Functions
    
    // swiftlint:disable identifier_name
    public override func hitTest(_ p: CGPoint) -> CALayer? {
        return nil
    }
    // swiftlint:enable identifier_name
    
    private func updateColors() {
        self.colors = self.gradientColors.map { $0.cgColor }
    }
    
}

📸 스크린샷

라이트 모드 다크 모드

static 변수를 사용하는 방식에서 FileManager를 사용하는 방식으로 리팩토링
여러 곳에 파편화되어 있던 여정 시작 / 중단 / 재개 로직을 Home 부분에 통합
@SwiftyJunnos SwiftyJunnos self-assigned this Jan 11, 2024
@SwiftyJunnos SwiftyJunnos added the 🎨 UI 비즈니스 로직이 없는 UI 작업 label Jan 11, 2024
@SwiftyJunnos SwiftyJunnos linked an issue Jan 11, 2024 that may be closed by this pull request
1 task
@PushedGun PushedGun added this to the 🏃‍♂️ 여정 기록 milestone Jan 12, 2024
Copy link
Member

@PushedGun PushedGun left a comment

Choose a reason for hiding this comment

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

수정해주신 내용 잘 보았습니다.

CAGradientLayer을 extension하여 사용하는 방식도 괜찮았을 것 같아요.
고생하셨습니다!

@SwiftyJunnos SwiftyJunnos merged commit a29b108 into iOS/release Jan 12, 2024
31 checks passed
@SwiftyJunnos SwiftyJunnos deleted the iOS/task/Gradient branch January 12, 2024 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 UI 비즈니스 로직이 없는 UI 작업
Projects
Status: ✅ 완료
Development

Successfully merging this pull request may close these issues.

지도 그라디언트 레이어 추가
2 participants