Skip to content

Conversation

@jboix
Copy link
Member

@jboix jboix commented Oct 23, 2025

Description

Resolves #78 by adding support for the metadata.live option in playlist items.

Changes Made

  • When metadata.live is true, the duration element now displays "Live" instead, and the vjs-card-duration-live class is added for styling purposes.

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.

@github-actions
Copy link

github-actions bot commented Oct 23, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 96.89% 4708 / 4859
🔵 Statements 96.89% 4708 / 4859
🔵 Functions 94.9% 335 / 353
🔵 Branches 88.25% 556 / 630
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/pillarbox-playlist/src/components/modal/pillarbox-playlist-menu-item.js 94.11% 80% 100% 94.11% 41-43
Generated in workflow #244 for commit 9a0196e by the Vitest Coverage Report Action

@github-actions
Copy link

github-actions bot commented Oct 23, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://SRGSSR.github.io/pillarbox-web-suite/pr-preview/pr-113/

Built to branch gh-pages at 2025-10-24 11:09 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Resolves #78 by adding support for the `metadata.live` option in playlist items.

When `metadata.live` is true, the duration element now displays "Live" instead,
and the `vjs-card-duration-live` class is added for styling purposes.
@jboix jboix force-pushed the feat/playlist-livestream-item branch from e93d007 to 9a0196e Compare October 24, 2025 11:07
@jboix jboix marked this pull request as ready for review October 24, 2025 11:23
@jboix jboix requested a review from amtins October 24, 2025 11:23
@jboix jboix self-assigned this Oct 24, 2025
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.

Implementing an option to define whether a **PlaylistItem** is a live stream, which will help identify live stream items in the playlist.

1 participant