From 20367b9421f67831795fbfbbaab45e25c9b57d23 Mon Sep 17 00:00:00 2001
From: "opensearch-trigger-bot[bot]"
 <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Date: Fri, 18 Oct 2024 19:43:56 -0700
Subject: [PATCH] [BUG] Metrics fixes (#2217) (#2218)

* updated release notes (#1997)



* disable unfunctioning expand on metrics canvas



---------




(cherry picked from commit 6edd8ef4d246afed7d7e1b00b94ef87817255a40)

Signed-off-by: sumukhswamy <sumukhhs@amazon.com>
Signed-off-by: Eric <menwe@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Sumukh Swamy <sumukhhs@amazon.com>
---
 .../visualization_container.test.tsx.snap     | 42 -------------------
 .../visualization_container.tsx               | 36 ----------------
 2 files changed, 78 deletions(-)

diff --git a/public/components/custom_panels/panel_modules/visualization_container/__tests__/__snapshots__/visualization_container.test.tsx.snap b/public/components/custom_panels/panel_modules/visualization_container/__tests__/__snapshots__/visualization_container.test.tsx.snap
index b855f47d64..38d69afad3 100644
--- a/public/components/custom_panels/panel_modules/visualization_container/__tests__/__snapshots__/visualization_container.test.tsx.snap
+++ b/public/components/custom_panels/panel_modules/visualization_container/__tests__/__snapshots__/visualization_container.test.tsx.snap
@@ -94,48 +94,6 @@ exports[`Visualization Container Component renders add visualization container 1
                   </EuiText>
                 </div>
               </EuiFlexItem>
-              <EuiFlexItem
-                className="visualization-action-button"
-                grow={false}
-              >
-                <div
-                  className="euiFlexItem euiFlexItem--flexGrowZero visualization-action-button"
-                >
-                  <EuiIcon
-                    data-test-subj="removeVisualizationButton"
-                    onClick={[Function]}
-                    type="crossInACircleFilled"
-                  >
-                    <EuiIconBeaker
-                      aria-hidden={true}
-                      className="euiIcon euiIcon--medium euiIcon-isLoading"
-                      data-test-subj="removeVisualizationButton"
-                      focusable="false"
-                      onClick={[Function]}
-                      role="img"
-                      style={null}
-                    >
-                      <svg
-                        aria-hidden={true}
-                        className="euiIcon euiIcon--medium euiIcon-isLoading"
-                        data-test-subj="removeVisualizationButton"
-                        focusable="false"
-                        height={16}
-                        onClick={[Function]}
-                        role="img"
-                        style={null}
-                        viewBox="0 0 16 16"
-                        width={16}
-                        xmlns="http://www.w3.org/2000/svg"
-                      >
-                        <path
-                          d="M5.277 10.088c.02.014.04.03.057.047.582.55 1.134.812 1.666.812.586 0 1.84-.293 3.713-.88L9 6.212V2H7v4.212l-1.723 3.876Zm-.438.987L3.539 14h8.922l-1.32-2.969C9.096 11.677 7.733 12 7 12c-.74 0-1.463-.315-2.161-.925ZM6 2H5V1h6v1h-1v4l3.375 7.594A1 1 0 0 1 12.461 15H3.54a1 1 0 0 1-.914-1.406L6 6V2Z"
-                        />
-                      </svg>
-                    </EuiIconBeaker>
-                  </EuiIcon>
-                </div>
-              </EuiFlexItem>
             </div>
           </EuiFlexGroup>
         </div>
diff --git a/public/components/custom_panels/panel_modules/visualization_container/visualization_container.tsx b/public/components/custom_panels/panel_modules/visualization_container/visualization_container.tsx
index 136de2beef..238d0edbc6 100644
--- a/public/components/custom_panels/panel_modules/visualization_container/visualization_container.tsx
+++ b/public/components/custom_panels/panel_modules/visualization_container/visualization_container.tsx
@@ -6,10 +6,8 @@
 import {
   EuiButton,
   EuiSmallButton,
-  EuiSmallButtonIcon,
   EuiCodeBlock,
   EuiContextMenuItem,
-  EuiContextMenuPanel,
   EuiFlexGroup,
   EuiFlexItem,
   EuiIcon,
@@ -20,7 +18,6 @@ import {
   EuiModalHeader,
   EuiModalHeaderTitle,
   EuiPanel,
-  EuiPopover,
   EuiSpacer,
   EuiText,
   EuiToolTip,
@@ -104,7 +101,6 @@ export const VisualizationContainer = ({
   onEditClick,
   cloneVisualization,
   showFlyout,
-  removeVisualization,
   catalogVisualization,
   inlineEditor,
   actionMenuType,
@@ -117,7 +113,6 @@ export const VisualizationContainer = ({
   const [visualizationData, setVisualizationData] = useState<Plotly.Data[]>([]);
   const [isLoading, setIsLoading] = useState(true);
   const [isError, setIsError] = useState({} as VizContainerError);
-  const onActionsMenuClick = () => setIsPopoverOpen((currPopoverOpen) => !currPopoverOpen);
   const closeActionsMenu = () => setIsPopoverOpen(false);
   const { http, pplService } = coreRefs;
   const { setToast } = useToast();
@@ -374,37 +369,6 @@ export const VisualizationContainer = ({
                 </EuiToolTip>
               </EuiText>
             </EuiFlexItem>
-            <EuiFlexItem grow={false} className="visualization-action-button">
-              {editMode ? (
-                <EuiIcon
-                  data-test-subj="removeVisualizationButton"
-                  type="crossInACircleFilled"
-                  onClick={() => {
-                    removeVisualization(visualizationId);
-                  }}
-                />
-              ) : (
-                <EuiPopover
-                  button={
-                    <EuiSmallButtonIcon
-                      aria-label="actionMenuButton"
-                      iconType="boxesHorizontal"
-                      onClick={onActionsMenuClick}
-                    />
-                  }
-                  isOpen={isPopoverOpen}
-                  closePopover={closeActionsMenu}
-                  anchorPosition="downLeft"
-                  panelPaddingSize="none"
-                >
-                  <EuiContextMenuPanel
-                    items={popoverPanel}
-                    data-test-subj="panelViz__popoverPanel"
-                    size="s"
-                  />
-                </EuiPopover>
-              )}
-            </EuiFlexItem>
           </EuiFlexGroup>
           {inlineEditor}
         </div>