mirror of
https://github.com/jellyfin/jellyfin-web
synced 2025-03-30 19:56:21 +00:00
refactor fullscreenManager suggested by JustAMan
This commit is contained in:
parent
02c5fd688f
commit
7a5d66c792
1 changed files with 4 additions and 9 deletions
|
@ -81,16 +81,11 @@ define(['events', 'dom', 'apphost', 'browser'], function (events, dom, appHost,
|
|||
|
||||
if (appHost.supports("fullscreenchange") && (browser.edgeUwp || -1 !== navigator.userAgent.toLowerCase().indexOf("electron"))) {
|
||||
function isTargetValid(target) {
|
||||
|
||||
if (dom.parentWithTag(target, ['BUTTON', 'INPUT', 'TEXTAREA'])) {
|
||||
return false;
|
||||
}
|
||||
|
||||
return true;
|
||||
return !dom.parentWithTag(target, ['BUTTON', 'INPUT', 'TEXTAREA']);
|
||||
}
|
||||
|
||||
|
||||
dom.addEventListener(window, 'dblclick', function (e) {
|
||||
|
||||
|
||||
if (isTargetValid(e.target)) {
|
||||
if (manager.isFullScreen()) {
|
||||
manager.exitFullscreen();
|
||||
|
@ -98,7 +93,7 @@ define(['events', 'dom', 'apphost', 'browser'], function (events, dom, appHost,
|
|||
manager.requestFullscreen();
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
}, {
|
||||
passive: true
|
||||
});
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue