prettier and some refactor

This commit is contained in:
CJ 2025-12-03 10:45:45 -06:00
parent 29cf90c59c
commit 2ee1bea720
3 changed files with 19 additions and 25 deletions

View file

@ -440,7 +440,11 @@ export const ScenePlayer: React.FC<IScenePlayerProps> = PatchComponent(
}; };
// empty deps - only init once // empty deps - only init once
// showAbLoopControls is necessary to re-init the player when the config changes // showAbLoopControls is necessary to re-init the player when the config changes
}, [uiConfig?.showAbLoopControls, uiConfig?.enableChromecast, interfaceConfig?.autostartVideo]); }, [
uiConfig?.showAbLoopControls,
uiConfig?.enableChromecast,
interfaceConfig?.autostartVideo,
]);
useEffect(() => { useEffect(() => {
const player = getPlayer(); const player = getPlayer();
@ -487,17 +491,6 @@ export const ScenePlayer: React.FC<IScenePlayerProps> = PatchComponent(
vrMenu.setShowButton(showButton); vrMenu.setShowButton(showButton);
}, [getPlayer, scene, vrTag]); }, [getPlayer, scene, vrTag]);
// Sync autostart button with config changes
useEffect(() => {
const player = getPlayer();
if (!player) return;
const autostartButton = player.autostartButton();
if (autostartButton) {
autostartButton.syncWithConfig(interfaceConfig?.autostartVideo ?? false);
}
}, [getPlayer, interfaceConfig?.autostartVideo]);
// Player event handlers // Player event handlers
useEffect(() => { useEffect(() => {
const player = getPlayer(); const player = getPlayer();
@ -862,11 +855,11 @@ export const ScenePlayer: React.FC<IScenePlayerProps> = PatchComponent(
sceneSaveActivity, sceneSaveActivity,
]); ]);
// Sync autostart button with config changes
useEffect(() => { useEffect(() => {
const player = getPlayer(); const player = getPlayer();
if (!player) return; if (!player) return;
async function updateAutoStart(enabled: boolean) { async function updateAutoStart(enabled: boolean) {
await updateInterfaceConfig({ await updateInterfaceConfig({
variables: { variables: {
@ -875,12 +868,16 @@ export const ScenePlayer: React.FC<IScenePlayerProps> = PatchComponent(
}, },
}, },
}); });
console.log("updated interface config");
} }
const autostartButton = player.autostartButton(); const autostartButton = player.autostartButton();
autostartButton.updateAutoStart = updateAutoStart; if (autostartButton) {
}, [getPlayer, updateInterfaceConfig]); autostartButton.syncWithConfig(
interfaceConfig?.autostartVideo ?? false
);
autostartButton.updateAutoStart = updateAutoStart;
}
}, [getPlayer, updateInterfaceConfig, interfaceConfig?.autostartVideo]);
useEffect(() => { useEffect(() => {
const player = getPlayer(); const player = getPlayer();

View file

@ -25,7 +25,7 @@ class AutostartButton extends videojs.getComponent("Button") {
private updateIcon() { private updateIcon() {
this.removeClass("vjs-icon-play-circle"); this.removeClass("vjs-icon-play-circle");
this.removeClass("vjs-icon-cancel"); this.removeClass("vjs-icon-cancel");
if (this.autostartEnabled) { if (this.autostartEnabled) {
this.addClass("vjs-icon-play-circle"); this.addClass("vjs-icon-play-circle");
this.controlText(this.localize("Auto-start enabled (click to disable)")); this.controlText(this.localize("Auto-start enabled (click to disable)"));
@ -50,10 +50,9 @@ class AutostartButton extends videojs.getComponent("Button") {
class AutostartButtonPlugin extends videojs.getPlugin("plugin") { class AutostartButtonPlugin extends videojs.getPlugin("plugin") {
private button: AutostartButton; private button: AutostartButton;
private autostartEnabled: boolean; private autostartEnabled: boolean;
updateAutoStart: (enabled: boolean) => Promise<void> = updateAutoStart: (enabled: boolean) => Promise<void> = () => {
() => { return Promise.resolve();
return Promise.resolve(); };
};
constructor(player: VideoJsPlayer, options?: IAutostartButtonOptions) { constructor(player: VideoJsPlayer, options?: IAutostartButtonOptions) {
super(player, options); super(player, options);
@ -122,4 +121,3 @@ declare module "video.js" {
} }
export default AutostartButtonPlugin; export default AutostartButtonPlugin;

View file

@ -122,7 +122,7 @@ $sceneTabWidth: 450px;
background-color: rgba(80, 80, 80, 0.9); background-color: rgba(80, 80, 80, 0.9);
border-radius: 50%; border-radius: 50%;
color: #fff; color: #fff;
content: "\f103"; content: "\f103";
font-size: 1em; font-size: 1em;
line-height: 1; line-height: 1;
margin-right: 1rem; margin-right: 1rem;
@ -151,7 +151,6 @@ $sceneTabWidth: 450px;
} }
} }
.vjs-touch-overlay .vjs-play-control { .vjs-touch-overlay .vjs-play-control {
z-index: 1; z-index: 1;
} }