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

Disable page layout when loading the player #1050

Merged

Conversation

dennis531
Copy link
Collaborator

Diese Änderungen verhindern das Anzeigen der Stud.IP-Seite beim Laden eines Videos, sodass der Übergang zwischen Stud.IP und Player geschmeidiger ablaufen soll.

Altes Verhalten (Dark-Theme im Browser):

Screencast.from.2024-11-05.13-35-24.webm

Altes Verhalten (Light-Theme im Browser):

Screencast.from.2024-11-05.13-37-56.webm

Neues Verhalten (Dark-Theme im Browser):

Screencast.from.2024-11-05.13-36-17.webm

Neues Verhalten (Light-Theme im Browser):

Screencast.from.2024-11-05.13-36-56.webm

Teilt gerne mit, ob ihr dieses Verhalten oder das alte Verhalten präferiert.

@dennis531 dennis531 added type:visual-clarity v:3 Everything related to the Stud.IP Opencast Plugin Version 3.x labels Nov 5, 2024
@tgloeggl
Copy link
Member

Du hast jetzt auch den loading-spinner mit rausgenommen. Hast du das einmal in der Courseware getestet? Dort wird direkt immer ein Redirect bei den Videos gemacht, und wenn da die Seite einfach nur leer bleibt sieht das evtl. etwas merkwürdig aus.

@tgloeggl tgloeggl self-requested a review November 13, 2024 14:07
<? else : ?>
<form class="default" action="<?= $launch_url ?>" name="ltiLaunchForm" id="ltiLaunchForm" method="post" encType="application/x-www-form-urlencoded">
<div class="oc--loading-spinner oc--loading-redirect">
Copy link
Member

Choose a reason for hiding this comment

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

Dieser "spinner" ist hauptsächlich für den Courseware-Block nützlich, deshalb sollte er meines erachtens da bleiben.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Screencast.from.2024-11-13.16-00-54.webm

Ich bevorzuge diese Lösung ohne den Spinner. Eventuell sieht diese Lösung nicht so gut aus, wenn das Laden des Videos länger dauert.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Screencast.from.2024-11-13.16-18-16.webm

Wenn das Internet langsamer ist, sehen Nutzende einfach einen weiße Seite, wenn das Video geladen wird. Die aktuelle Lösung behebt das Problem auch nicht:

Screencast.from.2024-11-13.16-22-45.webm

@tgloeggl tgloeggl merged commit d18b44c into elan-ev:master Nov 25, 2024
3 of 4 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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants