fix(palette): cancel hover timeout on click to prevent double-click needed to open palette

[KeerthiKumarR]

Fixes: #7495

Problem

When a palette (e.g. Meter) is open and the user clicks a different palette
(e.g. Ornament), the new palette does not open on the first click — a second
click is required.

Root Cause

_loadPaletteButtonHandler sets a 400ms hover timeout via _menuOpenTimeout
on onmouseover that calls showPalette. When the user moves the mouse from
one palette row to another and clicks, the hover timeout from the mouseover
fires and calls _hideMenus, racing with the click handler which also calls
showPalette. The net result is the first click only closes the open palette
without opening the new one.

The onmouseout handler already calls _clearPendingMenuOpen() to cancel
the timeout, but onclick did not — leaving the race condition open.

PR category

• Bug Fix

Fix

Add this._clearPendingMenuOpen() at the top of the onclick else-branch in
_loadPaletteButtonHandler, consistent with how onmouseout already handles it.

Files Changed

• js/palette.js — one line added in _loadPaletteButtonHandler

[github-actions]

🧪 Jest Test Results

✅ All Jest tests passed! This PR is ready to merge.

Coverage: Statements: 48.3% | Branches: 39.92% | Functions: 53.01% | Lines: 48.7%
Master Coverage: Statements: 48.29% | Branches: 39.91% | Functions: 52.99% | Lines: 48.69%

[KeerthiKumarR]

@walterbender @Ashutoshx7 Please have a look when you have time!

[sapnilbiswas]

lgtm!