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

Videos table #817

Merged
merged 4 commits into from
Oct 16, 2023
Merged

Videos table #817

merged 4 commits into from
Oct 16, 2023

Conversation

dennis531
Copy link
Collaborator

@dennis531 dennis531 commented Oct 14, 2023

Änderungen:

  • Ersetzt die Videoliste durch eine Stud.IP-Liste
  • Die Videos werden durch die Spaltenüberschriften sortiert, statt wie bisher über das Sortierendropdown
  • Tooltip beim Views-Indikator

Offene Fragen:

  • Aktuell wird in einer Playlist die letzte Sortierung aus der Playlistenbearbeitenansicht als Standard verwendet. Also wenn der Autor die Playlist aufsteigend nach Namen bearbeitet hat, wird diese Sortierung beim Aufruf der Playlist initial verwendet. Wollen wir das Verhalten beibehalten oder besser eine Standardsortierung (z.B. Benutzerdefiniert) festlegen?

Screenshots:
image

@alinastolzenburg @g550 Erstellt gerne Issues zu den Anforderungen, die ihr in #753 gestellt habt. Leider habe ich nicht alle umgesetzt, da manche eventuell noch Klärungsbedarf benötigen. Insbesondere die Beschreibungen könnten auf kleinen Bildschirmen zu viel Platz benötigen und die Bedienung beeinträchtigen.

Refs #753

@github-actions
Copy link

This pull request has conflicts ☹
Please resolve those so we can review the pull request.
Thanks.

@dennis531 dennis531 added v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x type:visual-clarity labels Oct 14, 2023
@alinastolzenburg
Copy link
Collaborator

alinastolzenburg commented Oct 16, 2023

@dennis531 Betreff Playlistensortierung: ja wollen wir so beibehalten :)

@tgloeggl
Copy link
Member

Die weiteren Änderungen aus #753 können dann in einem separaten Pull-Request einfließen.

@tgloeggl tgloeggl merged commit 056a55f into elan-ev:master Oct 16, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:visual-clarity v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x
Projects
Development

Successfully merging this pull request may close these issues.

3 participants