TUM-Dev / gocast

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

Settings Page UI #1152

Closed MatthiasReumann closed 11 months ago

MatthiasReumann commented 1 year ago

Motivation and Context

Update settings page UI for consistency with other pages.

Description

Steps for Testing

  1. Log in
  2. Navigate to settings page
  3. Update name, playback speed, etc.
  4. Should work
  5. 🚀

Screenshots

Screenshot 2023-09-07 at 07 45 38

github-actions[bot] commented 1 year ago

Your Testserver will be ready at https://1152.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 |
alexanderstephan commented 12 months ago

The UI itself looks really good. However, I noticed that the menu disappears in the upper right corner. Is that intended?

Reference: image

MatthiasReumann commented 12 months ago

The UI itself looks really good. However, I noticed that the menu disappears in the upper right corner. Is that intended?

Reference: image

Yup. Same for the info pages. I thought that it makes sense to minimize the footprint of the go-template (i.e. include .User object) in order to make it more easy to migrate to a REST-only approach.

alexanderstephan commented 12 months ago

Okay, I see 👍 In any case, people can just use the back button in their browser.