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

playlist scroll pos #1120

Merged
merged 4 commits into from
Sep 21, 2023
Merged

playlist scroll pos #1120

merged 4 commits into from
Sep 21, 2023

Conversation

mono424
Copy link
Collaborator

@mono424 mono424 commented Aug 15, 2023

Motivation and Context

If the playlist sidebar is not visible on page start, the scroll position is not set to the active item.

Bildschirmaufnahme.2023-09-21.um.12.06.14.mov

Description

This pr adds a observer that scrolls to the right position upon sidebar open.

Steps for Testing

Prerequisites:

  • 1 Students
  1. Log in
  2. Navigate to a Livestream
  3. Check if its scrolled to the right item if you open the playlist sidebar

Screenshots

Bildschirmaufnahme.2023-09-21.um.12.05.51.mov

@github-actions
Copy link

Your Testserver will be ready at https://1120.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

@joschahenningsen
Copy link
Member

What's the status of this? @mono424

@mono424 mono424 marked this pull request as ready for review September 21, 2023 10:07
@mono424
Copy link
Collaborator Author

mono424 commented Sep 21, 2023

What's the status of this? @mono424

Its ready to get reviewed :)

Copy link
Member

@joschahenningsen joschahenningsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Appears to be working just as intended. Thanks!

@joschahenningsen joschahenningsen merged commit ee92bd6 into dev Sep 21, 2023
8 checks passed
@joschahenningsen joschahenningsen deleted the feat/playlist-scroll-pos branch September 21, 2023 11:31
Mjaethers pushed a commit to Mjaethers/gocast that referenced this pull request Nov 3, 2023
* first test

* clearner code

* lint fi
SebiWrn pushed a commit that referenced this pull request May 7, 2024
* first test

* clearner code

* lint fi
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants