From 2faf43ef80d8745bfa70c87381f64f173975fd80 Mon Sep 17 00:00:00 2001
From: mathuo <6710312+mathuo@users.noreply.github.com>
Date: Tue, 16 Nov 2021 22:11:25 +0000
Subject: [PATCH] chore: fix sonar code smells

---
 packages/dockview/src/gridview/gridviewComponent.ts      | 4 ----
 packages/dockview/src/gridview/gridviewPanel.ts          | 1 -
 packages/dockview/src/groupview/groupview.ts             | 1 -
 packages/dockview/src/paneview/draggablePaneviewPanel.ts | 8 +++++---
 packages/dockview/src/splitview/core/splitview.ts        | 7 -------
 packages/dockview/src/splitview/splitviewComponent.ts    | 3 ---
 6 files changed, 5 insertions(+), 19 deletions(-)

diff --git a/packages/dockview/src/gridview/gridviewComponent.ts b/packages/dockview/src/gridview/gridviewComponent.ts
index 8bec7ab28..fbe96cb33 100644
--- a/packages/dockview/src/gridview/gridviewComponent.ts
+++ b/packages/dockview/src/gridview/gridviewComponent.ts
@@ -352,12 +352,8 @@ export class GridviewComponent
                     const group = groupItem.value;
                     if (group !== panel) {
                         group.setActive(false);
-                        // group.api._onDidActiveChange.fire({ isActive: false });
-                        // group.api.setActive(false);
                     } else {
                         group.setActive(true);
-                        // group.api._onDidActiveChange.fire({ isActive: true });
-                        // group.setActive(true);
                     }
                 });
             })
diff --git a/packages/dockview/src/gridview/gridviewPanel.ts b/packages/dockview/src/gridview/gridviewPanel.ts
index c0bb5288f..208940a0f 100644
--- a/packages/dockview/src/gridview/gridviewPanel.ts
+++ b/packages/dockview/src/gridview/gridviewPanel.ts
@@ -132,7 +132,6 @@ export abstract class GridviewPanel
             this.api.onVisibilityChange((event) => {
                 const { isVisible } = event;
                 const { containerApi } = this.params as GridviewInitParameters;
-                // this.api.setVisible(isVisible);
                 containerApi.setVisible(this, isVisible);
             }),
             this.api.onActiveChange(() => {
diff --git a/packages/dockview/src/groupview/groupview.ts b/packages/dockview/src/groupview/groupview.ts
index ac25c82f4..4e8a44c49 100644
--- a/packages/dockview/src/groupview/groupview.ts
+++ b/packages/dockview/src/groupview/groupview.ts
@@ -608,7 +608,6 @@ export class Groupview extends CompositeDisposable implements IGroupview {
             this.tabsContainer.hide();
             this.container.appendChild(this.watermark.element);
 
-            // this.contentContainer.openPanel(this.watermark);
             this.watermark.updateParentGroup(this.parent, true);
         }
         if (!this.isEmpty && this.watermark) {
diff --git a/packages/dockview/src/paneview/draggablePaneviewPanel.ts b/packages/dockview/src/paneview/draggablePaneviewPanel.ts
index d9b4ac7e1..e8d9a129e 100644
--- a/packages/dockview/src/paneview/draggablePaneviewPanel.ts
+++ b/packages/dockview/src/paneview/draggablePaneviewPanel.ts
@@ -1,7 +1,10 @@
 import { DragHandler } from '../dnd/abstractDragHandler';
-import { getPaneData, LocalSelectionTransfer } from '../dnd/dataTransfer';
+import {
+    getPaneData,
+    LocalSelectionTransfer,
+    PaneTransfer,
+} from '../dnd/dataTransfer';
 import { Droptarget, DroptargetEvent, Position } from '../dnd/droptarget';
-import { PaneTransfer } from '../dnd/dataTransfer';
 import { Emitter, Event } from '../events';
 import { IDisposable } from '../lifecycle';
 import { Orientation } from '../splitview/core/splitview';
@@ -10,7 +13,6 @@ import {
     PanePanelInitParameter,
     PaneviewPanel,
 } from './paneviewPanel';
-import { addClasses } from '../dom';
 
 interface ViewContainer {
     readonly title: string;
diff --git a/packages/dockview/src/splitview/core/splitview.ts b/packages/dockview/src/splitview/core/splitview.ts
index 6a139c1c5..6d46d6636 100644
--- a/packages/dockview/src/splitview/core/splitview.ts
+++ b/packages/dockview/src/splitview/core/splitview.ts
@@ -856,27 +856,20 @@ export class Splitview {
                     (position > 0 || this.startSnappingEnabled)
                 ) {
                     this.updateSash(sash, SashState.MINIMUM);
-                    // sash.state = SashState.Minimum;
                 } else if (
                     snappedAfter &&
                     collapsesDown[index] &&
                     (position < this.contentSize || this.endSnappingEnabled)
                 ) {
-                    // sash.state = SashState.Maximum;
                     this.updateSash(sash, SashState.MAXIMUM);
                 } else {
-                    // sash.state = SashState.Disabled;
                     this.updateSash(sash, SashState.DISABLED);
                 }
             } else if (min && !max) {
-                // sash.state = SashState.Minimum;
                 this.updateSash(sash, SashState.MINIMUM);
             } else if (!min && max) {
-                // sash.state = SashState.Maximum;
-
                 this.updateSash(sash, SashState.MAXIMUM);
             } else {
-                // sash.state = SashState.Enabled;
                 this.updateSash(sash, SashState.ENABLED);
             }
         }
diff --git a/packages/dockview/src/splitview/splitviewComponent.ts b/packages/dockview/src/splitview/splitviewComponent.ts
index 6b227975e..c1133167a 100644
--- a/packages/dockview/src/splitview/splitviewComponent.ts
+++ b/packages/dockview/src/splitview/splitviewComponent.ts
@@ -195,18 +195,15 @@ export class SplitviewComponent
         this.getPanels()
             .filter((v) => v !== view)
             .forEach((v) => {
-                // v.api._;
                 v.api._onDidActiveChange.fire({ isActive: false });
                 if (!skipFocus) {
                     v.focus();
                 }
-                // v.setActive(false, skipFocus);
             });
         view.api._onDidActiveChange.fire({ isActive: true });
         if (!skipFocus) {
             view.focus();
         }
-        // view.setActive(true, skipFocus);
     }
 
     getPanels(): SplitviewPanel[] {