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

fixed code smells

This commit is contained in:
vitorsemeano 2020-11-08 20:56:08 +00:00
parent fc9f747db2
commit 5c58d1472f
3 changed files with 65 additions and 59 deletions

View file

@ -144,7 +144,7 @@ export function show(button) {
} }
function showActivePlayerMenu(playerInfo) { function showActivePlayerMenu(playerInfo) {
showActivePlayerMenuInternal(dialogHelper, playerInfo); showActivePlayerMenuInternal(playerInfo);
} }
function disconnectFromPlayer(currentDeviceName) { function disconnectFromPlayer(currentDeviceName) {
@ -182,7 +182,7 @@ function disconnectFromPlayer(currentDeviceName) {
} }
} }
function showActivePlayerMenuInternal(dialogHelper, playerInfo) { function showActivePlayerMenuInternal(playerInfo) {
let html = ''; let html = '';
const dialogOptions = { const dialogOptions = {

View file

@ -1995,6 +1995,7 @@ export default function (view, params) {
function onWebSocketMessage(e, data) { function onWebSocketMessage(e, data) {
const msg = data; const msg = data;
const apiClient = getApiClient();
if (msg.MessageType === 'UserDataChanged' && currentItem && msg.Data.UserId == apiClient.getCurrentUserId()) { if (msg.MessageType === 'UserDataChanged' && currentItem && msg.Data.UserId == apiClient.getCurrentUserId()) {
const key = currentItem.UserData.Key; const key = currentItem.UserData.Key;
@ -2012,63 +2013,68 @@ export default function (view, params) {
let currentItem; let currentItem;
const self = this; const self = this;
const apiClient = getApiClient();
const btnResume = view.querySelector('.mainDetailButtons .btnResume'); function init() {
const btnPlay = view.querySelector('.mainDetailButtons .btnPlay'); const apiClient = getApiClient();
if (layoutManager.tv && !btnResume.classList.contains('hide')) {
btnResume.classList.add('fab');
btnResume.classList.add('detailFloatingButton');
} else if (layoutManager.tv && btnResume.classList.contains('hide')) {
btnPlay.classList.add('fab');
btnPlay.classList.add('detailFloatingButton');
}
view.querySelectorAll('.btnPlay'); const btnResume = view.querySelector('.mainDetailButtons .btnResume');
bindAll(view, '.btnPlay', 'click', onPlayClick); const btnPlay = view.querySelector('.mainDetailButtons .btnPlay');
bindAll(view, '.btnResume', 'click', onPlayClick); if (layoutManager.tv && !btnResume.classList.contains('hide')) {
bindAll(view, '.btnInstantMix', 'click', onInstantMixClick); btnResume.classList.add('fab');
bindAll(view, '.btnShuffle', 'click', onShuffleClick); btnResume.classList.add('detailFloatingButton');
bindAll(view, '.btnPlayTrailer', 'click', onPlayTrailerClick); } else if (layoutManager.tv && btnResume.classList.contains('hide')) {
bindAll(view, '.btnCancelSeriesTimer', 'click', onCancelSeriesTimerClick); btnPlay.classList.add('fab');
bindAll(view, '.btnCancelTimer', 'click', onCancelTimerClick); btnPlay.classList.add('detailFloatingButton');
bindAll(view, '.btnDownload', 'click', onDownloadClick);
view.querySelector('.detailImageContainer').addEventListener('click', onPosterClick);
view.querySelector('.trackSelections').addEventListener('submit', onTrackSelectionsSubmit);
view.querySelector('.btnSplitVersions').addEventListener('click', function () {
splitVersions(self, view, apiClient, params);
});
bindAll(view, '.btnMoreCommands', 'click', onMoreCommandsClick);
view.querySelector('.selectSource').addEventListener('change', function () {
renderVideoSelections(view, self._currentPlaybackMediaSources);
renderAudioSelections(view, self._currentPlaybackMediaSources);
renderSubtitleSelections(view, self._currentPlaybackMediaSources);
});
view.addEventListener('viewshow', function (e) {
const page = this;
libraryMenu.setTransparentMenu(true);
if (e.detail.isRestored) {
if (currentItem) {
appRouter.setTitle('');
renderTrackSelections(page, self, currentItem, true);
}
} else {
reload(self, page, params);
} }
Events.on(apiClient, 'message', onWebSocketMessage); view.querySelectorAll('.btnPlay');
Events.on(playbackManager, 'playerchange', onPlayerChange); bindAll(view, '.btnPlay', 'click', onPlayClick);
}); bindAll(view, '.btnResume', 'click', onPlayClick);
view.addEventListener('viewbeforehide', function () { bindAll(view, '.btnInstantMix', 'click', onInstantMixClick);
Events.off(apiClient, 'message', onWebSocketMessage); bindAll(view, '.btnShuffle', 'click', onShuffleClick);
Events.off(playbackManager, 'playerchange', onPlayerChange); bindAll(view, '.btnPlayTrailer', 'click', onPlayTrailerClick);
libraryMenu.setTransparentMenu(false); bindAll(view, '.btnCancelSeriesTimer', 'click', onCancelSeriesTimerClick);
}); bindAll(view, '.btnCancelTimer', 'click', onCancelTimerClick);
view.addEventListener('viewdestroy', function () { bindAll(view, '.btnDownload', 'click', onDownloadClick);
currentItem = null; view.querySelector('.detailImageContainer').addEventListener('click', onPosterClick);
self._currentPlaybackMediaSources = null; view.querySelector('.trackSelections').addEventListener('submit', onTrackSelectionsSubmit);
self.currentRecordingFields = null; view.querySelector('.btnSplitVersions').addEventListener('click', function () {
}); splitVersions(self, view, apiClient, params);
});
bindAll(view, '.btnMoreCommands', 'click', onMoreCommandsClick);
view.querySelector('.selectSource').addEventListener('change', function () {
renderVideoSelections(view, self._currentPlaybackMediaSources);
renderAudioSelections(view, self._currentPlaybackMediaSources);
renderSubtitleSelections(view, self._currentPlaybackMediaSources);
});
view.addEventListener('viewshow', function (e) {
const page = this;
libraryMenu.setTransparentMenu(true);
if (e.detail.isRestored) {
if (currentItem) {
appRouter.setTitle('');
renderTrackSelections(page, self, currentItem, true);
}
} else {
reload(self, page, params);
}
Events.on(apiClient, 'message', onWebSocketMessage);
Events.on(playbackManager, 'playerchange', onPlayerChange);
});
view.addEventListener('viewbeforehide', function () {
Events.off(apiClient, 'message', onWebSocketMessage);
Events.off(playbackManager, 'playerchange', onPlayerChange);
libraryMenu.setTransparentMenu(false);
});
view.addEventListener('viewdestroy', function () {
currentItem = null;
self._currentPlaybackMediaSources = null;
self.currentRecordingFields = null;
});
}
init();
} }

View file

@ -20,8 +20,8 @@ function getThemeStylesheetInfo(id) {
let theme; let theme;
if (id) { if (id) {
theme = themes.find(theme => { theme = themes.find(currentTheme => {
return theme.id === id; return currentTheme.id === id;
}); });
} }