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

[WIP] [iOS & tvOS] Playlists - Add / Create Views #1436

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

JPKribs
Copy link
Member

@JPKribs JPKribs commented Feb 17, 2025

Summary

Continues: #1428

Creates an "Add to Playlist" view that is accessible from the ItemView. Allows users to add an item to a playlist OR add an item to a new playlist. The latter creates the playlist at the same time as the item is added.

In-Progress

  1. [iOS & tvOS] Playlists - Library & Item View #1428 needs to wrap up first so I can re-use the ListItemViewModel and poster changes.
  2. The AddItemToPlaylistView is functional and the API works correctly to add items & create playlists. However, I don't love the UI.
  3. Do we want Adding to Playlists in the menu ellipsis at the top like I have it now or should this be with the other actions buttons?
  4. Does having the other items that exist in the playlist help or should that part be removed?

Video

iOS [WIP]:

Simulator.Screen.Recording.-.iPhone.16.Pro.-.2025-02-18.at.17.47.41.mp4

tvOS [Final]*

Simulator.Screen.Recording.-.Apple.TV.4K.3rd.generation.-.2025-02-18.at.17.15.05.mp4

@JPKribs JPKribs added enhancement New feature or request low-priority labels Feb 17, 2025
…dded/removed as well as updated since added = dismiss but removed != dismiss. Moved over poster work from other PR.
…rwise this looks just about perfect. iOS UI just looks bad. Will need to figure out the best way to present items in the playlist. Should be the last TODO. Otherwise, waiting on: jellyfin#1428
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request low-priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant