Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jboix
Copy link
Member

@jboix jboix commented Sep 18, 2025

Description

Adds support for specifying a custom component for playlist items via the itemComponentName option. This allows replacing the default PillarboxPlaylistMenuItem without subclassing the entire dialog modal.

Changes Made

  • Added itemComponentName as a configurable option for the playlist modal dialog.

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.

Adds support for specifying a custom component for playlist items via the `itemComponentName`
option. This allows replacing the default `PillarboxPlaylistMenuItem` without subclassing the entire
dialog modal.
@jboix jboix self-assigned this Sep 18, 2025
@jboix jboix requested a review from amtins September 18, 2025 13:06
@jboix jboix added this to Pillarbox Sep 18, 2025
@github-project-automation github-project-automation bot moved this to πŸ“‹ Backlog in Pillarbox Sep 18, 2025
@jboix jboix moved this from πŸ“‹ Backlog to 🍿 Code Review in Pillarbox Sep 18, 2025
Copy link

Coverage Report

Status Category Percentage Covered / Total
πŸ”΅ Lines 96.79% 4409 / 4555
πŸ”΅ Statements 96.79% 4409 / 4555
πŸ”΅ Functions 93.8% 318 / 339
πŸ”΅ Branches 88.81% 524 / 590
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/pillarbox-playlist/src/components/modal/pillarbox-playlist-modal.js 100% 100% 100% 100%
Generated in workflow #216 for commit c09e533 by the Vitest Coverage Report Action

Copy link

PR Preview Action v1.6.2

πŸš€ View preview at
https://SRGSSR.github.io/pillarbox-web-suite/pr-preview/pr-101/

Built to branch gh-pages at 2025-09-18 13:08 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🍿 Code Review
Development

Successfully merging this pull request may close these issues.

1 participant