mirrors/jellyfin-web
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin-web synced 2025-03-30 19:56:21 +00:00
Code Activity

Add container for skip button to make it focusable

Browse source 
FocusManager.isCurrentlyFocusableInternal doesn't work with
fixed elements.
This commit is contained in:
Dmitry Lyzo 2025-02-02 22:32:54 +03:00
parent 16a084b009
commit 2ab61b6d7b
2 changed files with 14 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/components/playback/skipbutton.scss
View file

@ -1,10 +1,17 @@
.skip-button-container {
position: fixed;
left: 0;
right: 0;
bottom: 18%;
pointer-events: none;
z-index: 10000;
}
.skip-button {
display: flex;
align-items: center;
position: fixed;
bottom: 18%;
right: 16%;
z-index: 10000;
margin-left: auto;
margin-right: 16%;
padding: 12px 20px;
color: black;
border: none;
@ -15,6 +22,7 @@
gap: 3px;
box-shadow: 7px 6px 15px -14px rgba(0, 0, 0, 0.65);
cursor: pointer;
pointer-events: auto;
}
@media (orientation: landscape) and (max-height: 500px) {

3
src/components/playback/skipsegment.ts
View file

@ -49,7 +49,8 @@ class SkipSegment extends PlaybackSubscriber {
if (!this.skipElement && this.currentSegment) {
let buttonHtml = '';
buttonHtml += '<button is="emby-button" class="skip-button hide skip-button-hidden"></button>';
// FIXME: Move skip button to the video OSD
buttonHtml += '<div class="skip-button-container"><button is="emby-button" class="skip-button hide skip-button-hidden"></button></div>';
document.body.insertAdjacentHTML('beforeend', buttonHtml);