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

Fix the Notes View tags filter #1857

Merged
merged 3 commits into from
Jul 3, 2023
Merged

Conversation

Auriga2
Copy link
Collaborator

@Auriga2 Auriga2 commented Jun 26, 2023

Fix the Notes View tags filter to update notes consistently upon user selection/deselection of tags

Prerequisites

  • Reviewed the checklist

  • Reviewed feedback from the "Sonar Cloud" bot. Note that you have to wait
    for the "CI / Unit Tests") to complete first. Failed Unit tests can be
    debugged by adding the label "verbose logging" to the GitHub PR.

Description of the Change

Due to the Platform.runLater() in the updateSelectedTagsCombo , the selected tag filters were not updating before the UI is updated in the updateNotesUI() . It ( Platform.runLater() ) doesn't seem to be required because no GUI is updated in that function, hence removing it fixed the issue.

Alternate Designs

Moving the entire updateNotesUI() inside a Platform.runLater() but it'd degrade the performance

Why Should This Be In Core?

Benefits

Possible Drawbacks

Verification Process

Open a graph and Notes View. Perform some activities on the graph such as select a node, delete it

Select Auto notes filter and see notes appear for those activities

Select/Deselect tags under the Select tag(s) filter and verify it is only displaying the notes with matching tags.

Applicable Issues

#1854

@sonarcloud
Copy link

sonarcloud bot commented Jun 29, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
No Duplication information No Duplication information

@antares1470 antares1470 merged commit f544103 into master Jul 3, 2023
@antares1470 antares1470 deleted the bugfix/notes_view_tags_filter branch July 3, 2023 01:05
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