Browse Source

chore(issue stream): Fix save-search tooltip (#16743)

Tooltip should say "saved search" instead of "filter list"
adhiraj 5 years ago
parent
commit
2ab6f74d2a

+ 2 - 2
src/sentry/static/sentry/app/views/issueList/createSavedSearchButton.jsx

@@ -94,14 +94,14 @@ class CreateSavedSearchButton extends React.Component {
     return (
       <Access organization={organization} access={['org:write']}>
         <Button
-          title={withTooltip ? t('Add to organization filter list') : null}
+          title={withTooltip ? t('Add to organization saved searches') : null}
           onClick={this.onToggle}
           data-test-id="save-current-search"
           size="zero"
           borderless
           containerDisplayMode="inline-flex"
           type="button"
-          aria-label={t('Add to organization filter list')}
+          aria-label={t('Add to organization saved searches')}
           icon="icon-add-to-list"
           iconSize="16px"
           className={buttonClassName}

+ 6 - 2
tests/js/spec/views/issueList/createSavedSearchButton.spec.jsx

@@ -85,7 +85,9 @@ describe('CreateSavedSearchButton', function() {
       });
       wrapper.setProps({organization: orgWithoutFeature});
 
-      const button = wrapper.find('button[aria-label="Add to organization filter list"]');
+      const button = wrapper.find(
+        'button[aria-label="Add to organization saved searches"]'
+      );
       expect(button).toHaveLength(1);
     });
 
@@ -95,7 +97,9 @@ describe('CreateSavedSearchButton', function() {
       });
       wrapper.setProps({organization: orgWithoutAccess});
 
-      const button = wrapper.find('button[aria-label="Add to organization filter list"]');
+      const button = wrapper.find(
+        'button[aria-label="Add to organization saved searches"]'
+      );
       expect(button).toHaveLength(0);
     });
   });