From ceec65fb37562fad556cf0edf1ff9f1a100e0e68 Mon Sep 17 00:00:00 2001 From: PeekabooSteam Date: Fri, 10 Mar 2023 13:39:40 +0000 Subject: [PATCH] Smartened the button toggle logic. Could be smarter, though. --- client/public/stylesheets/olympus.css | 11 +++++------ client/src/index.ts | 6 ++++-- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/client/public/stylesheets/olympus.css b/client/public/stylesheets/olympus.css index 5d983a96..5d24df02 100644 --- a/client/public/stylesheets/olympus.css +++ b/client/public/stylesheets/olympus.css @@ -372,6 +372,11 @@ nav.ol-panel> :last-child { } +.ol-panel .ol-group-button-toggle button.off::before { + background-image: url("/images/icons/square-regular.svg"); +} + + .highlight-primary { @@ -626,12 +631,6 @@ body[data-hide-naval] #unit-visibility-control-naval { } -[data-hide-blue] #coalition-visibility-control #coalition-visibility-control-blue::before, -[data-hide-red] #coalition-visibility-control #coalition-visibility-control-red::before, -[data-hide-neutral] #coalition-visibility-control #coalition-visibility-control-neutral::before { - background-image: url("/images/icons/square-regular.svg"); -} - .toggle { --width: 40px; --height: calc(var(--width) / 2); diff --git a/client/src/index.ts b/client/src/index.ts index 8cae197c..cbed8a33 100644 --- a/client/src/index.ts +++ b/client/src/index.ts @@ -75,11 +75,12 @@ function setup() { if ( triggerElement instanceof HTMLElement ) { const eventName:string = triggerElement.dataset.onClick || ""; - const params:string = triggerElement.dataset.onClickParams || "{}"; + let params = JSON.parse( triggerElement.dataset.onClickParams || "{}" ); + params._element = triggerElement; if ( eventName ) { document.dispatchEvent( new CustomEvent( eventName, { - detail: JSON.parse( params ) + detail: params } ) ); } @@ -130,6 +131,7 @@ function setup() { document.addEventListener( "toggleCoalitionVisibility", ( ev:CustomEventInit ) => { + ev.detail._element.classList.toggle( "off" ); document.body.toggleAttribute( "data-hide-" + ev.detail.coalition ); });