Bring the preventUserConfigOpen into line with the rest of the setting implementations that I like more

This commit is contained in:
Oliver 2025-12-06 23:23:48 -07:00
parent 8ca82c2cc1
commit f40d9f9381
6 changed files with 27 additions and 34 deletions

View file

@ -1,10 +1,7 @@
import { __ID } from "../consts.mjs"; import { __ID } from "../consts.mjs";
import { preventUserConfigOpen } from "../settings/preventUserConfigOpen.mjs";
import { startSidebarExpanded } from "../settings/startSidebarExpanded.mjs"; import { startSidebarExpanded } from "../settings/startSidebarExpanded.mjs";
Hooks.once(`init`, () => { Hooks.once(`init`, () => {
console.log(`${__ID} | Initializing`); console.log(`${__ID} | Initializing`);
preventUserConfigOpen.register();
startSidebarExpanded.register(); startSidebarExpanded.register();
}); });

View file

@ -1,12 +0,0 @@
import { preventUserConfigOpen } from "../settings/preventUserConfigOpen.mjs";
// #region Once
Hooks.once(`renderUserConfig`, (app, element) => {
// MARK: Prevent Unready Open
if (!game.ready && preventUserConfigOpen.value()) {
element.style.display = `none`;
app.close();
};
});
// #endregion Once

View file

@ -2,11 +2,12 @@
import "./hooks/init.mjs"; import "./hooks/init.mjs";
import "./hooks/ready.mjs"; import "./hooks/ready.mjs";
import "./hooks/renderSidebar.mjs"; import "./hooks/renderSidebar.mjs";
import "./hooks/renderUserConfig.mjs";
import { chatSidebarBackground } from "./settings/chatSidebarBackground.mjs"; import { chatSidebarBackground } from "./settings/chatSidebarBackground.mjs";
import { preventUserConfigOpen } from "./settings/preventUserConfigOpen.mjs";
import { startingSidebarTab } from "./settings/startingSidebarTab.mjs"; import { startingSidebarTab } from "./settings/startingSidebarTab.mjs";
Hooks.once(`init`, () => { Hooks.once(`init`, () => {
chatSidebarBackground(); chatSidebarBackground();
preventUserConfigOpen();
startingSidebarTab(); startingSidebarTab();
}); });

View file

@ -4,6 +4,7 @@ import { Logger } from "../utils/Logger.mjs";
const key = `chatSidebarBackground`; const key = `chatSidebarBackground`;
export function chatSidebarBackground() { export function chatSidebarBackground() {
Logger.log(`Registering setting: ${key}`);
// #region Registration // #region Registration
game.settings.register(__ID, key, { game.settings.register(__ID, key, {

View file

@ -1,8 +1,13 @@
import { __ID } from "../consts.mjs"; import { __ID } from "../consts.mjs";
import { Logger } from "../utils/Logger.mjs";
const key = `preventUserConfigOpen`; const key = `preventUserConfigOpen`;
const config = { export function preventUserConfigOpen() {
Logger.log(`Registering setting: ${key}`);
// #region Registration
game.settings.register(__ID, key, {
name: `OFT.setting.${key}.name`, name: `OFT.setting.${key}.name`,
hint: `OFT.setting.${key}.hint`, hint: `OFT.setting.${key}.hint`,
scope: `user`, scope: `user`,
@ -10,13 +15,15 @@ const config = {
default: false, default: false,
config: true, config: true,
requiresReload: false, requiresReload: false,
}; });
// #endregion Registration
export const preventUserConfigOpen = { // #region Implementation
value() { Hooks.once(`renderUserConfig`, (app, element) => {
return game.settings.get(__ID, key); if (!game.ready && game.settings.get(__ID, key)) {
}, element.style.display = `none`;
register() { app.close();
game.settings.register(__ID, key, config); };
}, });
// #endregion Implementation
}; };

View file

@ -4,7 +4,7 @@ import { Logger } from "../utils/Logger.mjs";
const key = `startingSidebarTab`; const key = `startingSidebarTab`;
export function startingSidebarTab() { export function startingSidebarTab() {
Logger.log(`Registering ${key} setting`); Logger.log(`Registering setting: ${key}`);
// #region Registration // #region Registration
game.settings.register(__ID, key, { game.settings.register(__ID, key, {
@ -49,4 +49,3 @@ export function startingSidebarTab() {
}); });
// #endregion Implementation // #endregion Implementation
}; };