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

grc-qt: Fixing incorrect error display #7287

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dl1ksv
Copy link
Contributor

@dl1ksv dl1ksv commented Apr 23, 2024

Description

Fixes #7284

In addition:

If on startup grc loads an invalid flowgraph, the fg is shown as valid, that is the error button is disabled.

If there are several tabs ar open that contain a mixture of valid and invalid fg's the error button is not adopted to the status of the selected tab.

This pr calculates the status of the buttons individually for each tab depending on it's fg.

Related Issue

Fixes #7284

Testing Done

Build a simple fg, save it, delete a connection. Save the fg again.
Restart grc --qt
The fg should display an error as a connection is missing.

Open several tabs with an mixture of correct and incorrect fg's and switch between the different tabs.
Watch the status of the 'errors' button

Checklist

Fixes gnuradio#7248

In addition:

If on startup grc loads an invalid flowgraph, the fg is shown as valid,
that is the error button is disabled.

If there are several tabs ar open that contain a mixture of valid and invalid fg's
the error button is not adopted to the status of the selected tab.

This pr calculates  the status of the buttons individually for each tab
depending on it's fg.

Signed-off-by: Volker Schroer <3470424+dl1ksv@users.noreply.github.com>
Signed-off-by: Volker Schroer <3470424+dl1ksv@users.noreply.github.com>
@dl1ksv
Copy link
Contributor Author

dl1ksv commented May 5, 2024

This commit Fixes: #7305

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

Successfully merging this pull request may close these issues.

grc-qt: a double clicking a connection deletes the connection but the fg stays valid
1 participant