Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IQ tool: disable unused controls when recording/playback is in progress. #1171

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vladisslav2011
Copy link
Contributor

Changing IO devices and loading/saving settings does not look like good thing to do while recording/playing an IQ file.
Disable seek slider during recording as it has no function in this state.
Disable file list and directory selector during recording/playback.

Demonstration: https://www.youtube.com/watch?v=fo0e3f7QuUQ

src/applications/gqrx/mainwindow.cpp Show resolved Hide resolved
src/qtgui/iq_tool.cpp Outdated Show resolved Hide resolved
src/qtgui/iq_tool.h Outdated Show resolved Hide resolved
@vladisslav2011 vladisslav2011 force-pushed the iq_tool_disable_controls branch from 036ea2b to 8ebe190 Compare November 17, 2022 21:43
src/qtgui/iq_tool.h Outdated Show resolved Hide resolved
@vladisslav2011 vladisslav2011 force-pushed the iq_tool_disable_controls branch from 8ebe190 to 063c7c3 Compare November 17, 2022 22:39
Copy link
Contributor

@pinkavaj pinkavaj left a comment

Choose a reason for hiding this comment

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

LGTM

Changing IO devices and loading/saving settings does not look like good thing
to do while recording/playing an IQ file.
Disable seek slider during recording as it has no function in this
state.
Disable file list and directory selector during recording/playback.
@vladisslav2011 vladisslav2011 force-pushed the iq_tool_disable_controls branch from 2ccb12f to 3c7fc73 Compare July 28, 2023 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants