Skip to content

Hide more DX Assistant context menu items when no spot is targeted - #1115

Merged
ea4k merged 1 commit into
masterfrom
claude/dx-assistant-spot-menu-5qnhhl
Aug 19, 2026
Merged

Hide more DX Assistant context menu items when no spot is targeted#1115
ea4k merged 1 commit into
masterfrom
claude/dx-assistant-spot-menu-5qnhhl

Conversation

@ea4k

@ea4k ea4k commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • Follow-up to the earlier fix that hid the "Spot" submenu when the DX Assistant context menu is opened without a spot under the cursor.
  • "Hide this spot"/"Hide these spots" and "Copy Callsign" are now left out of the menu entirely (instead of shown disabled) when there's no spot under the cursor.
  • "Clear all" is now left out when there's nothing to clear (no visible spots, no hidden calls, no worked QSOs this session).
  • "Show to map" is now left out when the table currently has no spots shown.

Test plan

  • Right-click on a row with a spot in the DX Assistant table and confirm "Hide this spot", "Copy Callsign", and "Spot" submenu still appear.
  • Right-click on empty table space (or the header) and confirm those items are absent, not just greyed out.
  • With an empty spot list, right-click and confirm "Clear all" / "Show to map" are absent; add spots and confirm they reappear.

Generated by Claude Code

Hide this spot, Copy Callsign, Clear all, and Show to map now follow
the same rule as the Spot submenu: they're left out of the DX
Assistant context menu when there is nothing for them to act on,
rather than shown greyed out.
@sonarqubecloud

Copy link
Copy Markdown

@ea4k
ea4k merged commit 79d455a into master Aug 19, 2026
5 of 8 checks passed
@ea4k
ea4k deleted the claude/dx-assistant-spot-menu-5qnhhl branch August 19, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants