TUM-Dev / gocast

TUMs lecture streaming service.
https://live.rbg.tum.de/
MIT License
177 stars 41 forks source link

1094 list view in new UI #1286

Closed SebiWrn closed 5 months ago

SebiWrn commented 6 months ago

Motivation and Context

Adds a List View to the course view.

Description

You can now switch between the thumbnail and list view in the course page.

Steps for Testing

Prerequisites:

  1. Log in
  2. Navigate to a Course
  3. Switch to list view

Screenshots

Default View: image

List View: image

github-actions[bot] commented 6 months ago

Your Testserver will be ready at https://1286.test.live.mm.rbg.tum.de in a few minutes.

Logins | Kurs1 | Kurs2 | Kurs3 | Kurs4 | | ---------------------------------------- | ------------------------- | ------------------------- | ------------------------- | | public | public | loggedin | enrolled | | prof1 | prof1 | prof2 | prof1
prof2 | | student1
student2
student3 | student1
student2 | student2
student3 | student1
student2 |
YiranDuan721 commented 5 months ago

Currently, the status of whether the list view is displayed is reset when the page is accessed again. I would prefer it to be preserved like the other options. I can see that the state "listView" is indeed preserved in the browser's local storage, but it doesn't look like it's being restored properly when the page loads or reloads.

SebiWrn commented 5 months ago

Thank you for your feedback, problem should be fixed now :)