From 165e86c96286a86f47af5595e01d1882fcd6d14c Mon Sep 17 00:00:00 2001 From: evanpelle Date: Wed, 2 Jul 2025 20:52:18 -0700 Subject: [PATCH] remove select on hover (#1330) ## Description: Hover no longer opens submenus ## 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: DISCORD_USERNAME --- src/client/graphics/layers/RadialMenu.ts | 30 ------------------------ 1 file changed, 30 deletions(-) diff --git a/src/client/graphics/layers/RadialMenu.ts b/src/client/graphics/layers/RadialMenu.ts index 199f28cf6..fa4a35b83 100644 --- a/src/client/graphics/layers/RadialMenu.ts +++ b/src/client/graphics/layers/RadialMenu.ts @@ -381,36 +381,6 @@ export class RadialMenu implements Layer { path.attr("filter", "url(#glow)"); path.attr("stroke-width", "3"); - const color = disabled - ? this.config.disabledColor - : d.data.color || "#333333"; - path.attr("fill", color); - - const subMenu = - this.params !== null ? d.data.subMenu?.(this.params) : null; - if ( - subMenu && - subMenu.length > 0 && - !disabled && - !( - this.currentLevel > 0 && - d.data.id === this.selectedItemId && - level === 0 - ) - ) { - if (this.submenuHoverTimeout !== null) { - window.clearTimeout(this.submenuHoverTimeout); - } - - // Set a small delay before opening submenu to prevent accidental triggers - this.submenuHoverTimeout = window.setTimeout(() => { - if (this.navigationInProgress) return; - this.navigationInProgress = true; - this.selectedItemId = d.data.id; - this.navigateToSubMenu(subMenu); - this.updateCenterButtonState("back"); - }, 200); - } }; const onMouseOut = (d: d3.PieArcDatum, path: any) => {