Fix issues reported by sonarqube linter

10 files changed, 24 insertions, 27 deletions

diff --git a/src/features/appearance/index.ts b/src/features/appearance/index.ts
index b6e944724..da75bf57b 100644
--- a/src/features/appearance/index.ts
+++ b/src/features/appearance/index.ts
@@ -165,12 +165,6 @@ function generateServiceRibbonWidthStyle(
       sidebarSizeBias = 44;
       break;
     }
-    case 55: {
-      fontSize = 11;
-      tabItemHeightBias = 13;
-      sidebarSizeBias = 37;
-      break;
-    }
     case 80: {
       fontSize = 11;
       tabItemHeightBias = 3;
diff --git a/src/features/todos/components/TodosWebview.tsx b/src/features/todos/components/TodosWebview.tsx
index 4b53df956..7b1b17e64 100644
--- a/src/features/todos/components/TodosWebview.tsx
+++ b/src/features/todos/components/TodosWebview.tsx
@@ -196,7 +196,7 @@ class TodosWebview extends Component<IProps, IState> {
         )}
         {isTodoUrlValid && (
           <Webview
-            // className={classes.webview} // TODO - [TS DEBT] style not found
+            // className={classes.webview} // TODO: [TS DEBT] style not found
             onDidAttach={() => {
               const { setTodosWebview } = this.props;
               setTodosWebview(this.webview);
diff --git a/src/features/todos/containers/TodosScreen.tsx b/src/features/todos/containers/TodosScreen.tsx
index 17f61bd95..8941a4b39 100644
--- a/src/features/todos/containers/TodosScreen.tsx
+++ b/src/features/todos/containers/TodosScreen.tsx
@@ -31,7 +31,7 @@ class TodosScreen extends Component<IProps> {
             this.props.stores!.services.isTodosServiceActive || false
           }
           isVisible={todosStore.isTodosPanelVisible}
-          // togglePanel={todoActions.toggleTodosPanel} // TODO - [TECH DEBT][PROP NOT USED IN COMPONENT] check it later
+          // togglePanel={todoActions.toggleTodosPanel} // TODO: [TECH DEBT][PROP NOT USED IN COMPONENT] check it later
           handleClientMessage={todoActions.handleClientMessage}
           setTodosWebview={webview => todoActions.setTodosWebview(webview)}
           width={todosStore.width}
diff --git a/src/features/todos/store.ts b/src/features/todos/store.ts
index 94be4db6a..882fa6a1e 100644
--- a/src/features/todos/store.ts
+++ b/src/features/todos/store.ts
@@ -149,7 +149,7 @@ export default class TodoStore extends FeatureStore {
   @action stop() {
     super.stop();
     debug('TodoStore::stop');
-    // this.reset(); // TODO - [TECH DEBT][PROP NOT IN CLASS] check it later
+    // this.reset(); // TODO: [TECH DEBT][PROP NOT IN CLASS] check it later
     this.isFeatureActive = false;
   }
 
diff --git a/src/features/webControls/components/WebControls.tsx b/src/features/webControls/components/WebControls.tsx
index 74b343948..e76fca6a2 100644
--- a/src/features/webControls/components/WebControls.tsx
+++ b/src/features/webControls/components/WebControls.tsx
@@ -34,10 +34,11 @@ const messages = defineMessages({
   },
 });
 
-const buttonTransition =
-  window && window.matchMedia('(prefers-reduced-motion: no-preference)')
-    ? 'opacity 0.25s'
-    : 'none';
+const buttonTransition = window?.matchMedia(
+  '(prefers-reduced-motion: no-preference)',
+)
+  ? 'opacity 0.25s'
+  : 'none';
 
 const styles = theme => ({
   root: {
@@ -211,7 +212,7 @@ class WebControls extends Component<IProps, IState> {
               });
               navigate(inputUrl);
 
-              if (this.inputRef && this.inputRef.current) {
+              if (this.inputRef?.current) {
                 this.inputRef.current.blur();
               }
             } else if (event.key === 'Escape') {
@@ -220,7 +221,7 @@ class WebControls extends Component<IProps, IState> {
                 inputUrl: url,
               });
 
-              if (this.inputRef && this.inputRef.current) {
+              if (this.inputRef?.current) {
                 this.inputRef.current.blur();
               }
             }
diff --git a/src/features/workspaces/components/WorkspaceDrawerItem.tsx b/src/features/workspaces/components/WorkspaceDrawerItem.tsx
index 0ad56d1ae..74ca2797d 100644
--- a/src/features/workspaces/components/WorkspaceDrawerItem.tsx
+++ b/src/features/workspaces/components/WorkspaceDrawerItem.tsx
@@ -19,10 +19,11 @@ const messages = defineMessages({
   },
 });
 
-const itemTransition =
-  window && window.matchMedia('(prefers-reduced-motion: no-preference)')
-    ? 'background-color 300ms ease-out'
-    : 'none';
+const itemTransition = window?.matchMedia(
+  '(prefers-reduced-motion: no-preference)',
+)
+  ? 'background-color 300ms ease-out'
+  : 'none';
 
 const styles = theme => ({
   item: {
diff --git a/src/features/workspaces/components/WorkspaceItem.tsx b/src/features/workspaces/components/WorkspaceItem.tsx
index b097a8298..6c4dd649b 100644
--- a/src/features/workspaces/components/WorkspaceItem.tsx
+++ b/src/features/workspaces/components/WorkspaceItem.tsx
@@ -1,4 +1,4 @@
-/* eslint-disable jsx-a11y/no-noninteractive-element-interactions */ // TODO - [TS DEBT] Need to check and remove it
+/* eslint-disable jsx-a11y/no-noninteractive-element-interactions */
 import { Component, ReactElement } from 'react';
 import { observer } from 'mobx-react';
 import withStyles, { WithStylesProps } from 'react-jss';
diff --git a/src/features/workspaces/components/WorkspaceSwitchingIndicator.tsx b/src/features/workspaces/components/WorkspaceSwitchingIndicator.tsx
index c9af22c96..e1cc0f785 100644
--- a/src/features/workspaces/components/WorkspaceSwitchingIndicator.tsx
+++ b/src/features/workspaces/components/WorkspaceSwitchingIndicator.tsx
@@ -14,10 +14,11 @@ const messages = defineMessages({
   },
 });
 
-const wrapperTransition =
-  window && window.matchMedia('(prefers-reduced-motion: no-preference)')
-    ? 'width 0.5s ease'
-    : 'none';
+const wrapperTransition = window?.matchMedia(
+  '(prefers-reduced-motion: no-preference)',
+)
+  ? 'width 0.5s ease'
+  : 'none';
 
 const styles = theme => ({
   wrapper: {
diff --git a/src/features/workspaces/components/WorkspacesDashboard.tsx b/src/features/workspaces/components/WorkspacesDashboard.tsx
index 06738b38c..c5f8909c1 100644
--- a/src/features/workspaces/components/WorkspacesDashboard.tsx
+++ b/src/features/workspaces/components/WorkspacesDashboard.tsx
@@ -147,7 +147,7 @@ class WorkspacesDashboard extends Component<IProps> {
                   icon="alert"
                   type="danger"
                   ctaLabel={intl.formatMessage(messages.tryReloadWorkspaces)}
-                  // ctaLoading={getUserWorkspacesRequest.isExecuting} //  TODO - [TECH DEBT][PROP NOT USED IN COMPONENT] need to check and update
+                  // ctaLoading={getUserWorkspacesRequest.isExecuting} // TODO: [TECH DEBT][PROP NOT USED IN COMPONENT] need to check and update
                   ctaOnClick={getUserWorkspacesRequest.retry}
                 >
                   {intl.formatMessage(messages.workspacesRequestFailed)}
diff --git a/src/features/workspaces/store.ts b/src/features/workspaces/store.ts
index d63feb1f8..53e3b62ed 100644
--- a/src/features/workspaces/store.ts
+++ b/src/features/workspaces/store.ts
@@ -28,7 +28,7 @@ export default class WorkspacesStore extends FeatureStore {
 
   @observable nextWorkspace: Workspace | undefined;
 
-  @observable workspaceBeingEdited: any = null; // TODO - [TS DEBT] fix type later
+  @observable workspaceBeingEdited: any = null; // TODO: [TS DEBT] fix type later
 
   @observable isSwitchingWorkspace = false;
 
@@ -36,7 +36,7 @@ export default class WorkspacesStore extends FeatureStore {
 
   @observable isSettingsRouteActive = false;
 
-  stores: any; // TODO - [TS DEBT] fix type later
+  stores: any; // TODO: [TS DEBT] fix type later
 
   actions: Actions | undefined;