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

Add solution #10

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

xavierLowmiller
Copy link

My solution

(Best viewed in the AirDropAnimationView_Previews)

I've identified four states that the view can be in:

  1. "Choose recipient", which shows the subtitle "from Amos" and is otherwise static
  2. "Waiting", which shows the subtitle "Waiting..." and pulses indefinitely
  3. "Sending", which shows the subtitle "Sending..." and animates the ring progress around the icon
  4. "Sent", which shows the subtitle "Sent" and is otherwise static

All in all, there's only two animations:

  1. The indefinite pulse in PulsatingText
  2. The animated ring in RingProgress

To show off the animations, I've implemented a function (showAnimations())
that simply iterates through the four phases after some seconds. In a real
implementation, the progress for the "Sending" phase should obviously come from
some network state, but for now it just takes two seconds (hard-coded).

Obviously, this could use a lot more polishing, but I really enjoyed breaking
down the problem into its phases and conquering each individually ✌️

Screen.Recording.2022-08-14.at.18.33.07.mov

@amosgyamfi
Copy link
Collaborator

Thanks for the submission, Xaver. Let's go through all the solutions tomorrow.

@DaemonLoki
Copy link
Contributor

I was so happy to see your submission Xaver! :)

Really excited to discuss it in tonight's live stream. :)

@xavierLowmiller
Copy link
Author

Thanks! I'm really looking forward to your feedback ✌️

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