Skip to content

Add ordering of the playlist in the TV demo#1033

Merged
StaehliJ merged 1 commit intoedit_playlist_demo_tvfrom
add_playlist_sorting
May 19, 2025
Merged

Add ordering of the playlist in the TV demo#1033
StaehliJ merged 1 commit intoedit_playlist_demo_tvfrom
add_playlist_sorting

Conversation

@MGaetan89
Copy link
Contributor

Pull request

Description

This PR adds the ability to sort the items of the playlist to the TV demo.

Changes made

  • Add a long-click event to each item of the playlist to start the move action.
  • When going up or down, the item is moved accordingly.

Checklist

  • APIs have been properly documented (if relevant).
  • The documentation has been updated (if relevant).
  • New unit tests have been written (if relevant).
  • The demo has been updated (if relevant).

@MGaetan89 MGaetan89 self-assigned this May 15, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox May 15, 2025
@MGaetan89 MGaetan89 moved this from 📋 Backlog to 🚧 In Progress in Pillarbox May 15, 2025
@github-actions
Copy link

Code Coverage

Overall Project 50.69% 🟢

There is no coverage information present for the Files changed

@MGaetan89 MGaetan89 marked this pull request as ready for review May 15, 2025 14:18
@MGaetan89 MGaetan89 moved this from 🚧 In Progress to 🍿 Code Review in Pillarbox May 15, 2025
@MGaetan89 MGaetan89 linked an issue May 15, 2025 that may be closed by this pull request
3 tasks
@MGaetan89 MGaetan89 requested a review from StaehliJ May 15, 2025 14:20
@MGaetan89 MGaetan89 added this to the TV milestone May 15, 2025
@StaehliJ StaehliJ merged commit be10245 into edit_playlist_demo_tv May 19, 2025
10 checks passed
@github-project-automation github-project-automation bot moved this from 🍿 Code Review to ✅ Done in Pillarbox May 19, 2025
@StaehliJ StaehliJ deleted the add_playlist_sorting branch May 19, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Add simple playlist management to the demo tv player

2 participants