Skip to content

Waterfall cursor fix for split mode.#311

Merged
Chris-AC9KH merged 1 commit into
JS8Call-improved:masterfrom
Chris-AC9KH:pr/split_fix
May 31, 2026
Merged

Waterfall cursor fix for split mode.#311
Chris-AC9KH merged 1 commit into
JS8Call-improved:masterfrom
Chris-AC9KH:pr/split_fix

Conversation

@Chris-AC9KH

Copy link
Copy Markdown
Collaborator

Explanation of issue:
When I added the frequency offset slider I had basically designed it so the frequency offset slider controls the waterfall cursor. At the same time, the waterfall cursor can also control the frequency offset slider. I had hooked it up for HB's, but failed to hook it up for split mode. This causes the frequency slider to display the correct offset, but the waterfall cursor did not move when transmitting in split mode.

Explanation of issue:
When I added the frequency offset slider I had basically designed it
so the frequency offset slider controls the waterfall cursor.
At the same time, the waterfall cursor can also control the frequency
offset slider. I had hooked it up for HB's, but failed to hook it up
for split mode. This causes the frequency slider to display the correct
offset, but the waterfall cursor did not move.

@rruchte rruchte left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Chris-AC9KH Chris-AC9KH merged commit 3783bf7 into JS8Call-improved:master May 31, 2026
8 checks passed
@Chris-AC9KH Chris-AC9KH deleted the pr/split_fix branch May 31, 2026 13:38
@Joe-K0OG

Joe-K0OG commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

This change may lead to a little confusion. When I look at the waterfall, I see where my cursor is in relation to other stations on the waterfall. If running split (either rig or fake-it), when I transmit, my location on the waterfall as seen by other stations is where I placed it before transmission. With this change, looking at the waterfall while I'm transmitting in split mode (if the cursor is outside the 1500-2000Hz range), my waterfall cursor jumps to where it is NOT really located relative to other stations on the waterfall. This is misleading. The cursor on the waterfall is used to display my location in relation to other stations, which location is produced on-air by the combination of dial frequency and offset as displayed in the upper-left corner of JS8Call.

I therefore wish to argue for reversion of this change so that the cursor does not jump on the waterfall while transmitting in split mode. @Chris-AC9KH Please consider reversion. I think it makes more sense the way it was.

73,
-Joe-
K0OG

@Chris-AC9KH

Copy link
Copy Markdown
Collaborator Author

It's easily reverted. Just one line of code. What does the old software do before I added the offset slider?

@Chris-AC9KH

Copy link
Copy Markdown
Collaborator Author

Ok, I see the old code did NOT move the waterfall cursor during split operation. For some reason, there was some folks on the forum that thought this is a "bug". The offset slider moved and indicated the actual offset, but the waterfall cursor didn't move so they thought something was wrong.

@Joe-K0OG

Joe-K0OG commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Yeah, as I remember, it has ALWAYS stayed put on the waterfall, from pre-1.0 days. This keeps the cursor in proper relation to other signals on the waterfall, so makes more sense to revert to the old behavior.

@Chris-AC9KH

Copy link
Copy Markdown
Collaborator Author

Ok, I'll just pull that line of code out and direct-push it to master. No PR needed for that

@Joe-K0OG

Joe-K0OG commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Thanks Chris!

Good evening & 73,
-Joe-
K0OG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants