Skip to content

Fix: Clear detail panels on severity filter change (AST-136035)#245

Open
cx-aniket-shinde wants to merge 3 commits intointegration/eclipse-bug-fixesfrom
bug/AST-136035
Open

Fix: Clear detail panels on severity filter change (AST-136035)#245
cx-aniket-shinde wants to merge 3 commits intointegration/eclipse-bug-fixesfrom
bug/AST-136035

Conversation

@cx-aniket-shinde
Copy link
Copy Markdown
Contributor

…an ID combo overflow

  • Hide resultViewComposite and attackVectorCompositePanel when filter changes so the description and attack vector windows no longer show stale content
  • Replace fixed widthHint=520 on scan ID combo with SWT.FILL/grabExcess layout so the combo is always visible without needing to maximize the window

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Describe the purpose of this PR along with any background information and the impacts of the proposed change.

References

Include supporting link to GitHub Issue/PR number

Testing

Describe how this change was tested. Be specific about anything not tested and reasons why. If this solution has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

…an ID combo overflow

- Hide resultViewComposite and attackVectorCompositePanel when filter changes
  so the description and attack vector windows no longer show stale content
- Replace fixed widthHint=520 on scan ID combo with SWT.FILL/grabExcess layout
  so the combo is always visible without needing to maximize the window

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the bug Something isn't working label Apr 14, 2026
@cx-aniket-shinde cx-aniket-shinde changed the title Fix AST-136035: Clear detail panels on severity filter change; fix sc… Fix: Clear detail panels on severity filter change (AST-136035) Apr 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 14, 2026

Logo
Checkmarx One – Scan Summary & Detailsf2c16991-cc59-4ddd-bf68-70d4efd34fc2


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 HIGH CVE-2026-24400 Maven-org.assertj:assertj-core-3.27.2
detailsRecommended version: 3.27.7
Description: AssertJ provides Fluent testing assertions for Java and the Java Virtual Machine (JVM). Starting in version 1.4.0 prior to 3.27.7 and 4.0.0-M1, an ...
Attack Vector: LOCAL
Attack Complexity: LOW
Vulnerable Package

@cx-aniket-shinde cx-aniket-shinde changed the base branch from main to integration/eclipse-bug-fixes April 14, 2026 13:11
cx-aniket-shinde and others added 2 commits April 14, 2026 18:57
…y is filtered out

Previously, toggling any severity filter always hid the description and
attack vector panels. Now the panels are only cleared when the currently
displayed item belongs to a severity that was just disabled. If the item's
severity is still active, the panels stay visible.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
…anges

- Capture expanded elements before clearing the model so the snapshot
  is accurate when restoring after refresh
- Pass expand=true for FILTER_CHANGED (keep GET_RESULTS at false) so
  previously expanded group nodes are restored after filtering instead
  of collapsing the entire tree

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant