Skip to content

Conversation

@PhilippeWeidmann
Copy link
Member

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds DotLottie animation support to the onboarding flow by introducing the dotlottie-ios dependency and replacing static illustrations with animated .lottie files for the authorization permission screens (contacts and notifications). The changes refactor the animation handling to support both legacy Airbnb Lottie and new DotLottie formats with theme switching based on color scheme and accent color.

  • Adds DotLottie iOS library dependency via branch reference
  • Introduces two new animated .lottie files for permission screens
  • Refactors animation color/theme handling to support multiple animation formats

Reviewed Changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Tuist/Package.swift Updates ios-onboarding dependency to use feat/dotlottie branch instead of version
Tuist/Package.resolved Adds dotlottie-ios dependency and updates related package versions
MailResources/Animations/notificationPermission.lottie Adds new animated notification permission illustration with theme support
MailResources/Animations/addressBookPermission.lottie Adds new animated address book permission illustration with theme support
Mail/Views/Onboarding/WaveView.swift Refactors color update logic to handle both Airbnb Lottie and DotLottie animations
Mail/Views/Onboarding/OnboardingView.swift Updates animation access to use enum pattern matching
Mail/Views/Onboarding/AuthorizationView.swift Adds theme switching extension and replaces static images with DotLottie animations

@PhilippeWeidmann PhilippeWeidmann force-pushed the feat/onboarding-animation branch from fcebc6a to b76fbb3 Compare November 19, 2025 10:50
@sonarqubecloud
Copy link

@PhilippeWeidmann PhilippeWeidmann force-pushed the feat/onboarding-animation branch 3 times, most recently from 9be2eaf to 8f4224e Compare January 6, 2026 14:47
@PhilippeWeidmann PhilippeWeidmann force-pushed the feat/onboarding-animation branch from 8f4224e to d62024e Compare January 7, 2026 07:07
@PhilippeWeidmann PhilippeWeidmann marked this pull request as ready for review January 7, 2026 07:46
@PhilippeWeidmann PhilippeWeidmann force-pushed the feat/onboarding-animation branch from af2acbf to 9ea834b Compare January 7, 2026 10:50
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 7, 2026

@valentinperignon valentinperignon merged commit cc7dd0b into master Jan 7, 2026
11 checks passed
@valentinperignon valentinperignon deleted the feat/onboarding-animation branch January 7, 2026 12:34
PhilippeWeidmann added a commit that referenced this pull request Jan 9, 2026
…talyst

This reverts commit cc7dd0b, reversing
changes made to f0946d9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants