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

* Fix AST-136035: Clear detail panels on severity filter change; fix scan 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>

* Fix AST-136035: Clear right panels only when displayed item's severity 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>

* Fix AST-136035: Preserve tree expansion state when severity filter changes

- 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>

---------

Co-authored-by: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

Author: cx-aniket-shinde

checkmarx-ast-eclipse-plugin/src/com/checkmarx/eclipse/views/CheckmarxView.java

@@ -160,6 +160,7 @@ public class CheckmarxView extends ViewPart implements EventHandler {
 	private Text commentText;
 	private DisplayModel rootModel;
 	private String selectedSeverity, selectedState;
+	private DisplayModel currentlyDisplayedItem;
 	private Button triageButton;
 	private SelectionAdapter triageButtonAdapter, codeBashingAdapter;
 	private Shell shell = PlatformUI.getWorkbench().getActiveWorkbenchWindow().getShell();
@@ -1082,8 +1083,7 @@ private void createScanIdComboBox(Composite parent) {
 		scanIdComboViewer.setContentProvider(ArrayContentProvider.getInstance());
 		scanIdComboViewer.setInput(new ArrayList<>());
 
-		GridData gridData = new GridData();
-		gridData.widthHint = 520;
+		GridData gridData = new GridData(SWT.FILL, SWT.CENTER, true, false);
 		scanIdComboViewer.getCombo().setLayoutData(gridData);
 
 		scanIdComboViewer.getCombo().addListener(SWT.DefaultSelection, new Listener() {
@@ -1386,6 +1386,7 @@ protected IStatus run(IProgressMonitor arg0) {
 
 							if (selectedItem.getResult() != null && selectedItem.getResult().getSimilarityId() != null) {
 								sync.asyncExec(() -> {
+									currentlyDisplayedItem = selectedItem;
 									createTriageSeverityAndStateCombos(selectedItem);
 									populateTriageChanges(selectedItem);
 									resultViewComposite.setVisible(true);
@@ -2473,6 +2474,8 @@ private Image findSeverityImage(DisplayModel model) {
 	private void listener(PluginListenerDefinition definition) {
 		switch (definition.getListenerType()) {
 			case FILTER_CHANGED:
+				updateResultsTree(definition.getResutls(), true);
+				break;
 			case GET_RESULTS:
 				updateResultsTree(definition.getResutls(), false);
 				break;
@@ -2489,9 +2492,15 @@ private void listener(PluginListenerDefinition definition) {
 
 	private void updateResultsTree(List<DisplayModel> results, boolean expand) {
 		sync.asyncExec(() -> {
+			if (currentlyDisplayedItem == null
+					|| currentlyDisplayedItem.getSeverity() == null
+					|| !FilterState.isSeverityEnabled(currentlyDisplayedItem.getSeverity())) {
+				resultViewComposite.setVisible(false);
+				attackVectorCompositePanel.setVisible(false);
+			}
+			Object[] expanded = resultsTree.getExpandedElements();
 			rootModel.children.clear();
 			rootModel.children.addAll(results);
-			Object[] expanded = resultsTree.getExpandedElements();
 			resultsTree.refresh();
 			if (expand) {
 				Set<String> expandedDMNames = new HashSet<>();