From 1e73293d4f522933a0159051f03b130818db1aa7 Mon Sep 17 00:00:00 2001 From: evanpelle Date: Mon, 14 Jul 2025 08:57:38 -0700 Subject: [PATCH] remove radial animation, fix back button (#1427) ## Description: Removed the animation from the radial menu, it now pops up instantly Also fixed bug where back button was grayed out ## Please complete the following: - [x] I have added screenshots for all UI updates - [x] I process any text displayed to the user through translateText() and I've added it to the en.json file - [x] I have added relevant tests to the test directory - [x] I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced - [x] I understand that submitting code with bugs that could have been caught through manual testing blocks releases and new features for all contributors ## Please put your Discord username so you can be contacted if a bug or regression is found: evan --- src/client/graphics/layers/RadialMenu.ts | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/client/graphics/layers/RadialMenu.ts b/src/client/graphics/layers/RadialMenu.ts index fa4a35b83..e905f2dea 100644 --- a/src/client/graphics/layers/RadialMenu.ts +++ b/src/client/graphics/layers/RadialMenu.ts @@ -237,10 +237,12 @@ export class RadialMenu implements Layer { .append("g") .attr("class", `menu-level-${level}`); - // Set initial animation styles + // Set initial animation styles only for submenus (level > 0) if (level === 0) { - menuGroup.style("opacity", 0.5).style("transform", "scale(0.2)"); + // Main menu appears immediately without animation + menuGroup.style("opacity", 1).style("transform", "scale(1)"); } else { + // Submenus get the expansion animation menuGroup.style("opacity", 0).style("transform", "scale(0.5)"); } @@ -857,6 +859,11 @@ export class RadialMenu implements Layer { } private isCenterButtonEnabled(): boolean { + // Back button should always be enabled when in submenu levels + if (this.currentLevel > 0) { + return true; + } + if (this.params && this.centerButtonElement) { return !this.centerButtonElement.disabled(this.params); }