From ef2ad80ad9db932bce54bb5f69384231edef8769 Mon Sep 17 00:00:00 2001
From: Ruben Thoms <ruben.thoms@ceetronsolutions.com>
Date: Thu, 30 Jan 2025 14:07:39 +0100
Subject: [PATCH] Removed unnecessary error comments

---
 .../lib/components/Dialog/components/DialogComponent.tsx  | 1 -
 react/src/lib/components/EdsIcon/EdsIcon.tsx              | 2 --
 react/src/lib/components/Menu/components/Group/Group.tsx  | 1 -
 react/src/lib/components/Menu/components/Icon/Icon.tsx    | 2 --
 .../lib/components/Menu/components/MenuBar/MenuBar.tsx    | 1 -
 .../components/WebvizDialogTitle/WebvizDialogTitle.tsx    | 1 -
 .../components/WebvizContentOverlay.tsx                   | 3 ---
 .../components/WebvizToolbarButton.tsx                    | 1 -
 .../lib/components/WebvizPluginTour/WebvizPluginTour.tsx  | 3 ---
 .../components/FullScreenMenu/full-screen-menu.tsx        | 1 -
 .../WebvizSettingsDrawer/WebvizSettingsDrawer.tsx         | 2 --
 .../components/AuthorDialog/author-dialog.tsx             | 8 ++++----
 .../components/PluginActions/plugin-actions.tsx           | 7 -------
 .../components/ViewList/view-list.tsx                     | 2 --
 .../components/ViewSelector/view-selector.tsx             | 2 --
 .../WebvizSettingsGroup/WebvizSettingsGroup.tsx           | 2 --
 .../components/WebvizViewElement/WebvizViewElement.tsx    | 4 ----
 17 files changed, 4 insertions(+), 39 deletions(-)

diff --git a/react/src/lib/components/Dialog/components/DialogComponent.tsx b/react/src/lib/components/Dialog/components/DialogComponent.tsx
index e2d6fbb7..59dd6355 100644
--- a/react/src/lib/components/Dialog/components/DialogComponent.tsx
+++ b/react/src/lib/components/Dialog/components/DialogComponent.tsx
@@ -126,7 +126,6 @@ export const DialogComponent: React.FC<DialogProps> = (props) => {
                         color: "#ccc",
                     }}
                 >
-                    { /* @ts-expect-error - this is a very weird bug */}
                     <Icon name="close" />
                 </IconButton>
             </DialogTitle>
diff --git a/react/src/lib/components/EdsIcon/EdsIcon.tsx b/react/src/lib/components/EdsIcon/EdsIcon.tsx
index 124a5c5d..7695ff8e 100644
--- a/react/src/lib/components/EdsIcon/EdsIcon.tsx
+++ b/react/src/lib/components/EdsIcon/EdsIcon.tsx
@@ -28,7 +28,6 @@ export const EdsIcon: React.FC<EdsIconProps> = (props) => {
                     `https://eds-storybook-react.azurewebsites.net/?path=/story/icons--preview.`
                 }
             >
-                { /* @ts-expect-error - this is a very weird bug */}
                 <Icon
                     id={props.id}
                     data={edsIcons.report}
@@ -42,7 +41,6 @@ export const EdsIcon: React.FC<EdsIconProps> = (props) => {
 
     return (
         <>
-        { /* @ts-expect-error - this is a very weird bug */}
         <Icon id={props.id} data={icon} size={props.size} color={props.color} />
         </>
     );
diff --git a/react/src/lib/components/Menu/components/Group/Group.tsx b/react/src/lib/components/Menu/components/Group/Group.tsx
index 7687970c..aa697c1f 100644
--- a/react/src/lib/components/Menu/components/Group/Group.tsx
+++ b/react/src/lib/components/Menu/components/Group/Group.tsx
@@ -73,7 +73,6 @@ export const Group: React.FC<GroupProps> = (props) => {
                             : "Collapse group"
                     }
                 >
-                    { /* @ts-expect-error - this is a very weird bug */}
                     <EdsIcon
                         name={
                             !collapsed || props.forceOpen
diff --git a/react/src/lib/components/Menu/components/Icon/Icon.tsx b/react/src/lib/components/Menu/components/Icon/Icon.tsx
index 68f643a3..22f52428 100644
--- a/react/src/lib/components/Menu/components/Icon/Icon.tsx
+++ b/react/src/lib/components/Menu/components/Icon/Icon.tsx
@@ -27,7 +27,6 @@ export const Icon: React.FC<IconProps> = (props) => {
                     `https://eds-storybook-react.azurewebsites.net/?path=/story/icons--preview.`
                 }
             >
-                { /* @ts-expect-error - this is a very weird bug */}
                 <EdsIcon
                     data={edsIcons.report}
                     color="hsla(0, 100%, 50%, 1)"
@@ -39,7 +38,6 @@ export const Icon: React.FC<IconProps> = (props) => {
 
     return (
         <>
-        { /* @ts-expect-error - this is a very weird bug */}
         <EdsIcon
             data={icon}
             color={props.active ? "#FF1243" : "#989898"}
diff --git a/react/src/lib/components/Menu/components/MenuBar/MenuBar.tsx b/react/src/lib/components/Menu/components/MenuBar/MenuBar.tsx
index 72973560..6a87f3c9 100644
--- a/react/src/lib/components/Menu/components/MenuBar/MenuBar.tsx
+++ b/react/src/lib/components/Menu/components/MenuBar/MenuBar.tsx
@@ -331,7 +331,6 @@ export const MenuBar = React.forwardRef<HTMLDivElement, MenuBarProps>(
                         variant="ghost_icon"
                         onClick={handleMenuButtonClick}
                     >
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="menu" title="Open menu" />
                     </Button>
                 </div>
diff --git a/react/src/lib/components/WebvizDialog/components/WebvizDialogTitle/WebvizDialogTitle.tsx b/react/src/lib/components/WebvizDialog/components/WebvizDialogTitle/WebvizDialogTitle.tsx
index b6c96465..7e7911ba 100644
--- a/react/src/lib/components/WebvizDialog/components/WebvizDialogTitle/WebvizDialogTitle.tsx
+++ b/react/src/lib/components/WebvizDialog/components/WebvizDialogTitle/WebvizDialogTitle.tsx
@@ -74,7 +74,6 @@ export const WebvizDialogTitle = React.forwardRef<
                         aria-label="close"
                         onClick={() => handleCloseClick()}
                     >
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="close" />
                     </IconButton>
                 </div>
diff --git a/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizContentOverlay.tsx b/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizContentOverlay.tsx
index ad078560..8bd7bbb0 100644
--- a/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizContentOverlay.tsx
+++ b/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizContentOverlay.tsx
@@ -43,14 +43,12 @@ const WebvizContentOverlay: React.FC<InferProps<typeof propTypes>> = ({
             <div className="webviz-plugin-data-owner">
                 {contactPerson && "name" in contactPerson && (
                     <p>
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="person" style={{ marginRight: "5px" }} />
                         {contactPerson.name}
                     </p>
                 )}
                 {contactPerson && "email" in contactPerson && (
                     <p>
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="email" style={{ marginRight: "5px" }} />
                         <a href="mailto:{this.props.contactPerson.email}">
                             {contactPerson.email}
@@ -59,7 +57,6 @@ const WebvizContentOverlay: React.FC<InferProps<typeof propTypes>> = ({
                 )}
                 {contactPerson && "phone" in contactPerson && (
                     <p>
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="call" style={{ marginRight: "5px" }} />
                         {contactPerson.phone}
                     </p>
diff --git a/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizToolbarButton.tsx b/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizToolbarButton.tsx
index e5d3c9d8..5784ece9 100644
--- a/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizToolbarButton.tsx
+++ b/react/src/lib/components/WebvizPluginPlaceholder/components/WebvizToolbarButton.tsx
@@ -24,7 +24,6 @@ const WebvizToolbarButton: React.FC<WebvizToolbarButtonProps> = ({
     target,
 }) => {
     const createIcon = (useOnClick: boolean) => (
-        // @ts-expect-error - this is a very weird bug
         <Icon
             data={icon}
             className={classNames({
diff --git a/react/src/lib/components/WebvizPluginTour/WebvizPluginTour.tsx b/react/src/lib/components/WebvizPluginTour/WebvizPluginTour.tsx
index 4d31f376..6e360527 100644
--- a/react/src/lib/components/WebvizPluginTour/WebvizPluginTour.tsx
+++ b/react/src/lib/components/WebvizPluginTour/WebvizPluginTour.tsx
@@ -277,7 +277,6 @@ export const WebvizPluginTour: React.FC<WebvizPluginTourProps> = (
                     >
                         {pluginData && (
                             <div className="WebvizPluginTour__View">
-                                { /* @ts-expect-error - this is a very weird bug */}
                                 <Icon name="view_carousel" />
                                 <span>
                                     {pluginData.views.find(
@@ -314,7 +313,6 @@ export const WebvizPluginTour: React.FC<WebvizPluginTourProps> = (
                                         }
                                     >
                                         Next
-                                        { /* @ts-expect-error - this is a very weird bug */}
                                         <Icon name="arrow_forward" />
                                     </Button>
                                 }
@@ -329,7 +327,6 @@ export const WebvizPluginTour: React.FC<WebvizPluginTourProps> = (
                                         }}
                                         disabled={currentTourStep === 0}
                                     >
-                                        { /* @ts-expect-error - this is a very weird bug */}
                                         <Icon name="arrow_back" />
                                         Back
                                     </Button>
diff --git a/react/src/lib/components/WebvizPluginsWrapper/components/FullScreenMenu/full-screen-menu.tsx b/react/src/lib/components/WebvizPluginsWrapper/components/FullScreenMenu/full-screen-menu.tsx
index 3d3a2a64..75494f08 100644
--- a/react/src/lib/components/WebvizPluginsWrapper/components/FullScreenMenu/full-screen-menu.tsx
+++ b/react/src/lib/components/WebvizPluginsWrapper/components/FullScreenMenu/full-screen-menu.tsx
@@ -38,7 +38,6 @@ export const FullScreenMenu: React.FC<FullScreenMenuProps> = (
                         }
                     >
                         <Tooltip title={action.tooltip}>
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <>{icon && <Icon data={icon} />}</>
                         </Tooltip>
                     </IconButton>
diff --git a/react/src/lib/components/WebvizSettingsDrawer/WebvizSettingsDrawer.tsx b/react/src/lib/components/WebvizSettingsDrawer/WebvizSettingsDrawer.tsx
index 00837330..f2df8cd3 100644
--- a/react/src/lib/components/WebvizSettingsDrawer/WebvizSettingsDrawer.tsx
+++ b/react/src/lib/components/WebvizSettingsDrawer/WebvizSettingsDrawer.tsx
@@ -164,9 +164,7 @@ export const WebvizSettingsDrawer: React.FC<WebvizSettingsDrawerProps> = (
                             }`}
                             onClick={() => handleToggleOpenClick()}
                         >
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <Icon name="chevron_left" />
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <Icon name="settings" />
                         </Button>
                     </Tooltip>
diff --git a/react/src/lib/components/WebvizSettingsDrawer/components/AuthorDialog/author-dialog.tsx b/react/src/lib/components/WebvizSettingsDrawer/components/AuthorDialog/author-dialog.tsx
index fce4726d..05756b0e 100644
--- a/react/src/lib/components/WebvizSettingsDrawer/components/AuthorDialog/author-dialog.tsx
+++ b/react/src/lib/components/WebvizSettingsDrawer/components/AuthorDialog/author-dialog.tsx
@@ -48,7 +48,7 @@ export const AuthorDialog: React.FC<AuthorDialogProps> = (
                         color: "#ccc",
                     }}
                 >
-                    { /* @ts-expect-error - this is a very weird bug */}
+                    
                     <Icon name="close" />
                 </IconButton>
             </DialogTitle>
@@ -57,7 +57,7 @@ export const AuthorDialog: React.FC<AuthorDialogProps> = (
                     <ListItem>
                         <ListItemAvatar>
                             <Avatar>
-                                { /* @ts-expect-error - this is a very weird bug */}
+                                
                                 <Icon name="person" />
                             </Avatar>
                         </ListItemAvatar>
@@ -66,7 +66,7 @@ export const AuthorDialog: React.FC<AuthorDialogProps> = (
                     <ListItem>
                         <ListItemAvatar>
                             <Avatar>
-                                { /* @ts-expect-error - this is a very weird bug */}
+                                
                                 <Icon name="email" />
                             </Avatar>
                         </ListItemAvatar>
@@ -75,7 +75,7 @@ export const AuthorDialog: React.FC<AuthorDialogProps> = (
                     <ListItem>
                         <ListItemAvatar>
                             <Avatar>
-                                { /* @ts-expect-error - this is a very weird bug */}
+                                
                                 <Icon name="phone" />
                             </Avatar>
                         </ListItemAvatar>
diff --git a/react/src/lib/components/WebvizSettingsDrawer/components/PluginActions/plugin-actions.tsx b/react/src/lib/components/WebvizSettingsDrawer/components/PluginActions/plugin-actions.tsx
index 1ccbd7ba..c84ce3a0 100644
--- a/react/src/lib/components/WebvizSettingsDrawer/components/PluginActions/plugin-actions.tsx
+++ b/react/src/lib/components/WebvizSettingsDrawer/components/PluginActions/plugin-actions.tsx
@@ -563,7 +563,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                 onClick={() => handleFullScreenClick()}
             >
                 <Tooltip title="Open active plugin in fullscreen mode">
-                    { /* @ts-expect-error - this is a very weird bug */}
                     <Icon name="fullscreen" />
                 </Tooltip>
             </div>
@@ -572,7 +571,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                 onClick={() => handleScreenShotClick()}
             >
                 <Tooltip title="Take a screenshot from active plugin">
-                    { /* @ts-expect-error - this is a very weird bug */}
                     <Icon name="camera" />
                 </Tooltip>
             </div>
@@ -582,7 +580,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                     onClick={handleDownloadClick}
                 >
                     <Tooltip title="Download data from active plugin">
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="download" />
                     </Tooltip>
                 </div>
@@ -599,7 +596,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                             badgeContent={numDeprecationWarnings}
                             color="primary"
                         >
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <Icon name="warning_outlined" />
                         </Badge>
                     </Tooltip>
@@ -611,7 +607,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                     onClick={() => setOpenAuthorDialog(true)}
                 >
                     <Tooltip title="View active plugin's author">
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="person" />
                     </Tooltip>
                 </div>
@@ -622,7 +617,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                     onClick={() => setTourIsOpen(true)}
                 >
                     <Tooltip title="Start a tour through the active plugin">
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="help" />
                     </Tooltip>
                 </div>
@@ -633,7 +627,6 @@ export const PluginActions: React.FC<PluginActionsProps> = (
                     onClick={() => openInNewTab(feedbackUrl)}
                 >
                     <Tooltip title="Report issue/give feedback on the active plugin">
-                        { /* @ts-expect-error - this is a very weird bug */}
                         <Icon name="comment_solid" />
                     </Tooltip>
                 </div>
diff --git a/react/src/lib/components/WebvizSettingsDrawer/components/ViewList/view-list.tsx b/react/src/lib/components/WebvizSettingsDrawer/components/ViewList/view-list.tsx
index e3935084..c1c374ea 100644
--- a/react/src/lib/components/WebvizSettingsDrawer/components/ViewList/view-list.tsx
+++ b/react/src/lib/components/WebvizSettingsDrawer/components/ViewList/view-list.tsx
@@ -115,7 +115,6 @@ export const ViewList: React.FC<ViewListProps> = (props: ViewListProps) => {
                                     <div>
                                         {el.views[0].id ===
                                             props.activeViewId && (<>
-                                            { /* @ts-expect-error - this is a very weird bug */}
                                             <Icon name="check" />
                                             </>
                                         )}
@@ -153,7 +152,6 @@ export const ViewList: React.FC<ViewListProps> = (props: ViewListProps) => {
                                             <div>
                                                 {view.id ===
                                                     props.activeViewId && (<>
-                                                    { /* @ts-expect-error - this is a very weird bug */}
                                                     <Icon name="check" />
                                                     </>
                                                 )}
diff --git a/react/src/lib/components/WebvizSettingsDrawer/components/ViewSelector/view-selector.tsx b/react/src/lib/components/WebvizSettingsDrawer/components/ViewSelector/view-selector.tsx
index ffd3b764..253485c7 100644
--- a/react/src/lib/components/WebvizSettingsDrawer/components/ViewSelector/view-selector.tsx
+++ b/react/src/lib/components/WebvizSettingsDrawer/components/ViewSelector/view-selector.tsx
@@ -146,7 +146,6 @@ export const ViewSelector: React.FC<ViewSelectorProps> = (
                 onClick={() => setMenuOpen(true)}
             >
                 <div ref={viewCarouselRef} className="WebvizViewSelector__Icon">
-                    { /* @ts-expect-error - this is a very weird bug */}
                     <Icon name="view_carousel" />
                 </div>
                 <div
@@ -165,7 +164,6 @@ export const ViewSelector: React.FC<ViewSelectorProps> = (
                         width: isCollapsed ? 0 : "auto",
                     }}
                 >
-                    { /* @ts-expect-error - this is a very weird bug */}
                     <Icon name="chevron_down" />
                 </div>
             </div>
diff --git a/react/src/lib/components/WebvizSettingsGroup/WebvizSettingsGroup.tsx b/react/src/lib/components/WebvizSettingsGroup/WebvizSettingsGroup.tsx
index f2ee134b..b2055324 100644
--- a/react/src/lib/components/WebvizSettingsGroup/WebvizSettingsGroup.tsx
+++ b/react/src/lib/components/WebvizSettingsGroup/WebvizSettingsGroup.tsx
@@ -148,7 +148,6 @@ export const WebvizSettingsGroup: React.FC<WebvizSettingsGroupProps> = (
                 >
                     {props.viewId === "" && (
                         <div className="WebvizSettingsGroup__GlobalIcon">
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <Icon
                                 name="world"
                                 title="Global settings group"
@@ -162,7 +161,6 @@ export const WebvizSettingsGroup: React.FC<WebvizSettingsGroupProps> = (
                     {!props.alwaysOpen && (
                         <div>
                             <IconButton>
-                                { /* @ts-expect-error - this is a very weird bug */}
                                 <Icon
                                     name={
                                         props.open === true
diff --git a/react/src/lib/components/WebvizViewElement/WebvizViewElement.tsx b/react/src/lib/components/WebvizViewElement/WebvizViewElement.tsx
index 2074c16e..65730d35 100644
--- a/react/src/lib/components/WebvizViewElement/WebvizViewElement.tsx
+++ b/react/src/lib/components/WebvizViewElement/WebvizViewElement.tsx
@@ -553,7 +553,6 @@ export const WebvizViewElement: React.FC<WebvizViewElementProps> = (props) => {
                             <IconButton
                                 onClick={() => handleOpenSettingsDialog()}
                             >
-                                 { /* @ts-expect-error - this is a very weird bug */}
                                 <Icon name="settings" size={16} />
                             </IconButton>
                         </Tooltip>
@@ -564,7 +563,6 @@ export const WebvizViewElement: React.FC<WebvizViewElementProps> = (props) => {
                     <div>
                         <Tooltip title="Download data from view element">
                             <IconButton onClick={() => handleDownloadClick()}>
-                                { /* @ts-expect-error - this is a very weird bug */}
                                 <Icon name="download" size={16} />
                             </IconButton>
                         </Tooltip>
@@ -573,7 +571,6 @@ export const WebvizViewElement: React.FC<WebvizViewElementProps> = (props) => {
                 <div>
                     <Tooltip title="Take screenshot">
                         <IconButton onClick={handleScreenShotClick}>
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <Icon name="camera" size={16} />
                         </IconButton>
                     </Tooltip>
@@ -581,7 +578,6 @@ export const WebvizViewElement: React.FC<WebvizViewElementProps> = (props) => {
                 <div>
                     <Tooltip title="View in fullscreen">
                         <IconButton onClick={handleFullScreenClick}>
-                            { /* @ts-expect-error - this is a very weird bug */}
                             <Icon name="fullscreen" size={16} />
                         </IconButton>
                     </Tooltip>