Skip to content

[BUG 660] POTA & SOTA panels truncate information#672

Merged
accius merged 1 commit intoaccius:Stagingfrom
alanhargreaves:activate-ref-display
Mar 4, 2026
Merged

[BUG 660] POTA & SOTA panels truncate information#672
accius merged 1 commit intoaccius:Stagingfrom
alanhargreaves:activate-ref-display

Conversation

@alanhargreaves
Copy link
Copy Markdown
Collaborator

What does this PR do?

Updates ActivatePanel to only show the spot.ref in the reference column, but populates the title with "spot.ref - spot.name".

There has been some confusion about how I previously implemented this by putting the reference and name in the displayed column, and letting it display an ellipsis so that mousing over would display the reference and name.

This is a cleaner way of doing it.

Type of change

  • Bug fix
  • New feature
  • Performance improvement
  • Refactor / code cleanup
  • Documentation
  • Translation
  • Map layer plugin

How to test

  1. View an ActivatePanel and verify that we only see the reference and no ellipsis in the reference column
  2. Mouse over the reference
  3. Verify that the popup displays the reference and name

Checklist

  • App loads without console errors
  • Tested in Dark, Light, and Retro themes
  • Responsive at different screen sizes (desktop + mobile)
  • If touching server.js: caches have TTLs and size caps (we serve 2,000+ concurrent users)
  • If adding an API route: includes caching and error handling
  • If adding a panel: wired into Modern, Classic, and Dockable layouts
  • No hardcoded colors — uses CSS variables (var(--accent-cyan), etc.)
  • No .bak, .old, console.log debug lines, or test scripts included

Screenshots (if visual change)

Before:
image

After:
image

…able, but "spot.ref - spot.name" in the title (which is displayed in mouse-over).
@trevlarson-ORE
Copy link
Copy Markdown

@alanhargreaves - Thanks for the additional work on this. In Chromium (ver. 144.0.7559.109), mouse over isn't producing a pop-up with any information. I guess I'll wait for this PR to get merged into main and test again. Thanks!

@alanhargreaves
Copy link
Copy Markdown
Collaborator Author

That is peculiar, I wonder if it is a browser option to enable the title field to pop up. I was considering modifying the Weather popup to contain this information, but the weather popup only works for local installs.

@alanhargreaves
Copy link
Copy Markdown
Collaborator Author

Fixes: #660

@accius accius merged commit 279a30b into accius:Staging Mar 4, 2026
@alanhargreaves alanhargreaves deleted the activate-ref-display branch March 5, 2026 08:57
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.

3 participants