Skip to content

Added loop modes, play/pause, content mode#11

Merged
marcprux merged 5 commits intoskiptools:mainfrom
vincentborko:main
Jan 11, 2026
Merged

Added loop modes, play/pause, content mode#11
marcprux merged 5 commits intoskiptools:mainfrom
vincentborko:main

Conversation

@vincentborko
Copy link
Contributor

@vincentborko vincentborko commented Jan 11, 2026

  • Added MotionLoopMode enum:

    • playOnce - play once and stop
    • loop - loop forever (default)
    • autoReverse - back-forth loop forever
    • repeat(Int) - loop N times
    • repeatBackwards(Int) - back-forth N times
  • Added isPlaying parameter (default: true) for paused option

  • Added ContentMode for .fit and .fill (scaledToFill()) doesnt work

  • Added example views for showcase-fuse tracking, linespacing, mask, minimumscalefactor (for text) skipapp-showcase-fuse#34

Skip Pull Request Checklist:

  • REQUIRED: I have signed the Contributor Agreement
  • REQUIRED: I have tested my change locally with swift test
  • OPTIONAL: I have tested my change on an iOS simulator or device
  • OPTIONAL: I have tested my change on an Android emulator or device

Added MotionLoopMode enum:
playOnce - play once and stop
loop - loop forever (default)
autoReverse - back-forth loop forever
repeat(Int) - loop N times
repeatBackwards(Int) - back-forth N times

Added isPlaying parameter to MotionView (default: true)
@cla-bot cla-bot bot added the cla-signed label Jan 11, 2026
@marcprux
Copy link
Member

It looks like this has some conflicts with #9 (see https://github.com/skiptools/skip-motion/pull/11/conflicts). I can merge them together, unless you would prefer to?

@vincentborko
Copy link
Contributor Author

if you could do it, it would be great. I dont know why that happened - and I dont know well how merge conflicts get resolved

@marcprux
Copy link
Member

Thanks again!

@marcprux marcprux merged commit 54bf6e2 into skiptools:main Jan 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants