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

feat(playlist-plugin): introduce repeat one mode #22

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Conversation

jboix
Copy link
Contributor

@jboix jboix commented Jun 5, 2024

Description

Resolves #19 by introducing a new "repeat one" mode to the playlist plugin.

The UI has been modified to include a toggle button that cycles through three modes: no repeat, repeat all, and repeat one. See the following example:

repeat-toggle-demo.mov

Changes Made

BREAKING CHANGE:

  • Changed the API to toggle the repeat mode and the initialisation options.
  • The repeat mode is no longer a boolean. It is now a number representing each mode (0: no repeat, 1: repeat all, 2: repeat one).

Checklist

  • I have followed the project's style and contribution guidelines.
  • I have performed a self-review of my own changes.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.

Copy link

github-actions bot commented Jun 5, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-08-08 08:07 UTC

Copy link

github-actions bot commented Jun 5, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 99.59% 1224 / 1229
🔵 Statements 99.59% 1224 / 1229
🔵 Functions 97.46% 77 / 79
🔵 Branches 93.7% 119 / 127
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
packages/pillarbox-playlist/src/pillarbox-playlist-ui.js 100% 100% 100% 100%
packages/pillarbox-playlist/src/pillarbox-playlist.js 98.97% 88.88% 96.29% 98.97% 155-158, 448
packages/pillarbox-playlist/src/components/pillarbox-playlist-button.js 100% 100% 100% 100%
packages/pillarbox-playlist/src/components/modal/pillarbox-playlist-menu-item.js 100% 100% 100% 100%
packages/pillarbox-playlist/src/components/modal/pillarbox-playlist-next-item-button.js 100% 100% 100% 100%
packages/pillarbox-playlist/src/components/modal/pillarbox-playlist-repeat-button.js 100% 90% 100% 100%
packages/pillarbox-playlist/src/components/modal/pillarbox-playlist-shuffle-button.js 100% 100% 100% 100%
packages/pillarbox-playlist/src/components/modal/pillarbox-previous-item-button.js 100% 100% 100% 100%
Generated in workflow #93

@jboix jboix force-pushed the feat/repeat-one branch 2 times, most recently from 83d6619 to 0ccacac Compare June 5, 2024 18:54
@MGaetan89 MGaetan89 linked an issue Jun 13, 2024 that may be closed by this pull request
Copy link
Member

@amtins amtins left a comment

Choose a reason for hiding this comment

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

I'll take another look when the merge conflicts are resolved and the PR is no longer in draft.

packages/pillarbox-playlist/src/lang/de.json Outdated Show resolved Hide resolved
@jboix jboix force-pushed the feat/repeat-one branch from 0ccacac to 3dcea40 Compare July 12, 2024 08:57
- Introduced a new "repeat one" mode to the playlist plugin.
- Modified the UI to include a toggle button that cycles through three modes: no repeat, repeat all,
  and repeat one.

BREAKING CHANGE:

- Changed the API to toggle the repeat mode and the initialization options.
- The repeat mode is no longer a boolean. It is now a number representing each mode (0: no repeat,
  1: repeat all, 2: repeat one).
@jboix jboix force-pushed the feat/repeat-one branch from 3dcea40 to 7005629 Compare July 12, 2024 09:08
@jboix jboix marked this pull request as ready for review July 12, 2024 09:20
@jboix jboix requested a review from amtins July 12, 2024 09:50
Updated the JSDoc type reference for the `Button` component to the correct
path in the video.js library.
Copy link
Member

@amtins amtins left a comment

Choose a reason for hiding this comment

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

LGTM!

@jboix jboix merged commit 57abd71 into main Aug 8, 2024
4 checks passed
@jboix jboix deleted the feat/repeat-one branch August 8, 2024 08:07
Copy link

github-actions bot commented Aug 8, 2024

🎉 This PR is included in version @srgssr/pillarbox-playlist-v2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[playlist-plugin] Add "Repeat One" mode
2 participants