FreeTubeApp / FreeTube

An Open Source YouTube app for privacy
https://freetubeapp.io/
GNU Affero General Public License v3.0
13.65k stars 864 forks source link

Fix 'shaka-video-container' class being removed from video container #6090

Closed OothecaPickle closed 2 weeks ago

OothecaPickle commented 3 weeks ago

Fix 'shaka-video-container' class being removed from video container

Pull Request Type

Related issue

further resolves #5838

Description

When Vue adds dynamic classes to the video container for full window mode or forcing a 16:9 aspect ratio, the shaka-video-container class is inadvertently removed from the video container element.

Because the upstream CSS that hides the cursor is applied to .shaka-video-container.no-cursor, this results in the cursor no longer hiding when viewing 4:3 aspect ratio videos or after toggling full window mode on any video.

Screenshots

Testing

Before fix: View a 4:3 aspect ratio video or toggle full window mode on any video, see that theshaka-video-container class is improperly removed from the video container element and that the cursor is not properly hidden.

After fix: View a 4:3 aspect ratio video or toggle full window mode on any video, see that theshaka-video-container class remains and that the cursor is properly hidden.

Desktop

Additional context