Open
Conversation
This adds the titular table and everything necessary to show all playlists. On its own not of much use, apart from looking at the playlist names. Creating, editing, deleting etc. will be added in a later commit. Needs https://github.com/owi92/opencast/tree/playlist-admin-ui-api to work.
Features metadata and ACL display and editing. Todo: Figure out UI/UX for adding and removing entries.
I think this project should really get away from scss or at least reconsider and rewrite some styles. Why are there `first-child` selectors with `float: right`? That means you have to write column layouts all backwards, which is unintuitive and confusing. Anyway. This just removes some styles/rules from the `SaveEditFooter` component that would not only hide but completely remove certain buttons from the dom while they're disabled. That can lead to layout shifts and confused margins whenever they do pop in. So with this change, they are now always visible and simply use disabled styles and behaviour when appropriate.
Like most of these playlist related additions, this is heavily based on the structure and components that are used for events and series. I'm sure there is a lot of potential for deduplication, but I don't want to touch too many non-playlist files. I think that should be done in a dedicated PR.
This let users add and removes entries to and from their playlists using a GUI. The UI reuses the drag n drop approach that is present for editing table view columns. Entries are shown with some additional metadata, so they are easier to tell apart if some names are repeated (hopefully).
Events don't remove themselves automatically from playlists when they are deleted. Previously I would just show their UUID, but that isn't very helpful. This will now show an "unknown entry" note, which is slightly better I suppose.
Contributor
|
Use Run test server using develop.opencast.org as backend: Specify a different backend like stable.opencast.org: It may take a few seconds for the interface to spin up. |
Contributor
|
This pull request is deployed at test.admin-interface.opencast.org/1537/2026-03-22_16-57-37/ . |
9 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This adds the necessary interface elements that are needed to use playlists in the admin interface.
Structure and UI design follows what is already present for events and series, and reuses components whenever possible.
A notable addition is the modal tab for entries management, where users can add, reorder, and remove playlist entries.
This needs opencast/opencast#7496 to work properly.
I realize that the timing on this is a little unfortunate because of https://github.com/orgs/opencast/discussions/7493,
but I've been working on this for a while and wanted to get it out before the next dev meeting.
Closes #1399
Closes opencast/opencast#6909