fixing bug (#62577)

elastic · Apr 7, 2020 · 0458626 · 0458626
1 parent dc013cb
commit 0458626
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 11 deletions.
diff --git a/x-pack/legacy/plugins/apm/public/components/app/Settings/CustomizeUI/CustomLink/Title.tsx b/x-pack/legacy/plugins/apm/public/components/app/Settings/CustomizeUI/CustomLink/Title.tsx
@@ -28,7 +28,7 @@ export const Title = () => (
                 'xpack.apm.settings.customizeUI.customLink.info',
                 {
                   defaultMessage:
-                    "These links will be shown in the 'Actions' context menu for the transaction detail."
+                    'These links will be shown in the Actions context menu for transactions.'
                 }
               )}
             />

diff --git a/x-pack/legacy/plugins/apm/public/components/app/Settings/CustomizeUI/index.tsx b/x-pack/legacy/plugins/apm/public/components/app/Settings/CustomizeUI/index.tsx
@@ -14,8 +14,8 @@ export const CustomizeUI = () => {
     <>
       <EuiTitle size="l">
         <h1>
-          {i18n.translate('xpack.apm.settings.customizeUI', {
-            defaultMessage: 'Customize UI'
+          {i18n.translate('xpack.apm.settings.customizeApp', {
+            defaultMessage: 'Customize app'
           })}
         </h1>
       </EuiTitle>

diff --git a/x-pack/legacy/plugins/apm/public/components/app/Settings/index.tsx b/x-pack/legacy/plugins/apm/public/components/app/Settings/index.tsx
@@ -57,8 +57,8 @@ export const Settings: React.FC = props => {
                     isSelected: pathname === '/settings/apm-indices'
                   },
                   {
-                    name: i18n.translate('xpack.apm.settings.customizeUI', {
-                      defaultMessage: 'Customize UI'
+                    name: i18n.translate('xpack.apm.settings.customizeApp', {
+                      defaultMessage: 'Customize app'
                     }),
                     id: '3',
                     href: getAPMHref('/settings/customize-ui', search),

diff --git a/...gacy/plugins/apm/public/components/shared/TransactionActionMenu/CustomLink/index.test.tsx b/...gacy/plugins/apm/public/components/shared/TransactionActionMenu/CustomLink/index.test.tsx
@@ -28,7 +28,7 @@ describe('Custom links', () => {
     );
 
     expectTextsInDocument(component, [
-      'No custom links found. Set up your own custom links i.e. a link to a specific Dashboard or external link.'
+      'No custom links found. Set up your own custom links, e.g., a link to a specific Dashboard or external link.'
     ]);
     expectTextsNotInDocument(component, ['Create']);
   });

diff --git a/...ck/legacy/plugins/apm/public/components/shared/TransactionActionMenu/CustomLink/index.tsx b/...ck/legacy/plugins/apm/public/components/shared/TransactionActionMenu/CustomLink/index.tsx
@@ -55,7 +55,7 @@ export const CustomLink = ({
       <EuiText size="xs" grow={false} style={{ width: px(300) }}>
         {i18n.translate('xpack.apm.customLink.empty', {
           defaultMessage:
-            'No custom links found. Set up your own custom links i.e. a link to a specific Dashboard or external link.'
+            'No custom links found. Set up your own custom links, e.g., a link to a specific Dashboard or external link.'
         })}
       </EuiText>
       <EuiSpacer size="s" />

diff --git a/...gacy/plugins/apm/public/components/shared/TransactionActionMenu/TransactionActionMenu.tsx b/...gacy/plugins/apm/public/components/shared/TransactionActionMenu/TransactionActionMenu.tsx
@@ -85,7 +85,13 @@ export const TransactionActionMenu: FunctionComponent<Props> = ({
     urlParams
   });
 
+  const closePopover = () => {
+    setIsActionPopoverOpen(false);
+    setIsCustomLinksPopoverOpen(false);
+  };
+
   const toggleCustomLinkFlyout = () => {
+    closePopover();
     setIsCustomLinkFlyoutOpen(isOpen => !isOpen);
   };
 
@@ -111,10 +117,7 @@ export const TransactionActionMenu: FunctionComponent<Props> = ({
       )}
       <ActionMenu
         id="transactionActionMenu"
-        closePopover={() => {
-          setIsActionPopoverOpen(false);
-          setIsCustomLinksPopoverOpen(false);
-        }}
+        closePopover={closePopover}
         isOpen={isActionPopoverOpen}
         anchorPosition="downRight"
         button={