Skip to content

Fix CW filter low-cut drag and VFO line thickness (#764)#888

Merged
ten9876 merged 1 commit into
aethersdr:mainfrom
AetherClaude:aetherclaude/issue-764
Apr 6, 2026
Merged

Fix CW filter low-cut drag and VFO line thickness (#764)#888
ten9876 merged 1 commit into
aethersdr:mainfrom
AetherClaude:aetherclaude/issue-764

Conversation

@AetherClaude

Copy link
Copy Markdown
Collaborator

Summary

Fixes #764

  • Delta-based filter edge dragging: Store an anchor offset when the user grabs a filter edge, so the edge tracks mouse movement rather than snapping to the absolute cursor position. This prevents the CW low-cut from jumping to 0 Hz on the first drag event.
  • Overlapping grab zone disambiguation: When both filter edges are within the 5px grab zone (common in CW's narrow filters), pick the closer edge instead of always defaulting to the low edge.
  • VFO center line width: Reduce the VFO center line from 2px to 1px when a filter edge is within 4px, preventing the visual "thick line" overlap in CW mode.

Test plan

  • In CW mode, grab the low-cut filter edge on the panadapter and drag — it should track smoothly without snapping to the carrier
  • In CW mode at narrow zoom, verify that clicking near overlapping filter edges grabs the intended (closer) edge
  • Verify the VFO center line appears thinner in CW mode where it overlaps with filter edges
  • Verify SSB/AM filter edge dragging still works normally (no regression)
  • Verify the VFO center line remains 2px wide in SSB/AM modes where filter edges are far from carrier

🤖 Generated with Claude Code

…rsdr#764)

- Store drag anchor offset on mouse press so filter edges don't snap to
  cursor position (delta-based dragging like FilterPassbandWidget)
- Disambiguate overlapping grab zones by picking the closer edge
- Reduce VFO center line width when a filter edge is within 4px

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aethersdr-agent aethersdr-agent Bot mentioned this pull request Apr 6, 2026
2 tasks
@ten9876 ten9876 marked this pull request as ready for review April 6, 2026 20:59
@ten9876 ten9876 self-requested a review as a code owner April 6, 2026 20:59
@ten9876 ten9876 merged commit 5db89d7 into aethersdr:main Apr 6, 2026
5 checks passed
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.

CW loc cut slope

2 participants