Browse Source

ref(searchbar): Remove max searchbar items due to new default state and set max height (#36214)

Jenn Mueng 2 years ago
parent
commit
e2a1881cd5

+ 1 - 5
static/app/views/dashboardsV2/widgetBuilder/buildSteps/filterResultsStep/eventsSearchBar.tsx

@@ -4,10 +4,7 @@ import SearchBar, {SearchBarProps} from 'sentry/components/events/searchBar';
 import {MAX_QUERY_LENGTH} from 'sentry/constants';
 import {MAX_QUERY_LENGTH} from 'sentry/constants';
 import {Organization, PageFilters, SavedSearchType} from 'sentry/types';
 import {Organization, PageFilters, SavedSearchType} from 'sentry/types';
 import {WidgetQuery} from 'sentry/views/dashboardsV2/types';
 import {WidgetQuery} from 'sentry/views/dashboardsV2/types';
-import {
-  MAX_MENU_HEIGHT,
-  MAX_SEARCH_ITEMS,
-} from 'sentry/views/dashboardsV2/widgetBuilder/utils';
+import {MAX_MENU_HEIGHT} from 'sentry/views/dashboardsV2/widgetBuilder/utils';
 
 
 interface Props {
 interface Props {
   onBlur: SearchBarProps['onBlur'];
   onBlur: SearchBarProps['onBlur'];
@@ -37,7 +34,6 @@ export function EventsSearchBar({
       onBlur={onBlur}
       onBlur={onBlur}
       useFormWrapper={false}
       useFormWrapper={false}
       maxQueryLength={MAX_QUERY_LENGTH}
       maxQueryLength={MAX_QUERY_LENGTH}
-      maxSearchItems={MAX_SEARCH_ITEMS}
       maxMenuHeight={MAX_MENU_HEIGHT}
       maxMenuHeight={MAX_MENU_HEIGHT}
       savedSearchType={SavedSearchType.EVENT}
       savedSearchType={SavedSearchType.EVENT}
     />
     />

+ 1 - 5
static/app/views/dashboardsV2/widgetBuilder/buildSteps/filterResultsStep/issuesSearchBar.tsx

@@ -9,10 +9,7 @@ import {getUtcDateString} from 'sentry/utils/dates';
 import useApi from 'sentry/utils/useApi';
 import useApi from 'sentry/utils/useApi';
 import withIssueTags from 'sentry/utils/withIssueTags';
 import withIssueTags from 'sentry/utils/withIssueTags';
 import {WidgetQuery} from 'sentry/views/dashboardsV2/types';
 import {WidgetQuery} from 'sentry/views/dashboardsV2/types';
-import {
-  MAX_MENU_HEIGHT,
-  MAX_SEARCH_ITEMS,
-} from 'sentry/views/dashboardsV2/widgetBuilder/utils';
+import {MAX_MENU_HEIGHT} from 'sentry/views/dashboardsV2/widgetBuilder/utils';
 import IssueListSearchBar from 'sentry/views/issueList/searchBar';
 import IssueListSearchBar from 'sentry/views/issueList/searchBar';
 
 
 interface Props {
 interface Props {
@@ -59,7 +56,6 @@ function IssuesSearchBarContainer({
           placeholder={t('Search for issues, status, assigned, and more')}
           placeholder={t('Search for issues, status, assigned, and more')}
           tagValueLoader={tagValueLoader}
           tagValueLoader={tagValueLoader}
           onSidebarToggle={() => undefined}
           onSidebarToggle={() => undefined}
-          maxSearchItems={MAX_SEARCH_ITEMS}
           savedSearchType={SavedSearchType.ISSUE}
           savedSearchType={SavedSearchType.ISSUE}
           dropdownClassName={css`
           dropdownClassName={css`
             max-height: ${MAX_MENU_HEIGHT}px;
             max-height: ${MAX_MENU_HEIGHT}px;

+ 1 - 5
static/app/views/dashboardsV2/widgetBuilder/buildSteps/filterResultsStep/releaseSearchBar.tsx

@@ -10,10 +10,7 @@ import {t} from 'sentry/locale';
 import {Organization, PageFilters, SavedSearchType, Tag, TagValue} from 'sentry/types';
 import {Organization, PageFilters, SavedSearchType, Tag, TagValue} from 'sentry/types';
 import useApi from 'sentry/utils/useApi';
 import useApi from 'sentry/utils/useApi';
 import {WidgetQuery} from 'sentry/views/dashboardsV2/types';
 import {WidgetQuery} from 'sentry/views/dashboardsV2/types';
-import {
-  MAX_MENU_HEIGHT,
-  MAX_SEARCH_ITEMS,
-} from 'sentry/views/dashboardsV2/widgetBuilder/utils';
+import {MAX_MENU_HEIGHT} from 'sentry/views/dashboardsV2/widgetBuilder/utils';
 
 
 import {SESSION_STATUSES, SESSIONS_FILTER_TAGS} from '../../releaseWidget/fields';
 import {SESSION_STATUSES, SESSIONS_FILTER_TAGS} from '../../releaseWidget/fields';
 
 
@@ -92,7 +89,6 @@ export function ReleaseSearchBar({
           onSearch={onSearch}
           onSearch={onSearch}
           onBlur={onBlur}
           onBlur={onBlur}
           maxQueryLength={MAX_QUERY_LENGTH}
           maxQueryLength={MAX_QUERY_LENGTH}
-          maxSearchItems={MAX_SEARCH_ITEMS}
           searchSource="widget_builder"
           searchSource="widget_builder"
           query={widgetQuery.conditions}
           query={widgetQuery.conditions}
           savedSearchType={SavedSearchType.SESSION}
           savedSearchType={SavedSearchType.SESSION}

+ 0 - 3
static/app/views/dashboardsV2/widgetBuilder/utils.tsx

@@ -313,9 +313,6 @@ export function getMetricFields(queries: WidgetQuery[]) {
   }, [] as string[]);
   }, [] as string[]);
 }
 }
 
 
-// Used to limit the number of results of the "filter your results" fields dropdown
-export const MAX_SEARCH_ITEMS = 5;
-
 // Used to set the max height of the smartSearchBar menu
 // Used to set the max height of the smartSearchBar menu
 export const MAX_MENU_HEIGHT = 250;
 export const MAX_MENU_HEIGHT = 250;
 
 

+ 1 - 1
static/app/views/issueList/searchBar.tsx

@@ -133,7 +133,6 @@ class IssueListSearchBar extends Component<Props, State> {
       <SmartSearchBar
       <SmartSearchBar
         searchSource="main_search"
         searchSource="main_search"
         hasRecentSearches
         hasRecentSearches
-        maxSearchItems={5}
         savedSearchType={SavedSearchType.ISSUE}
         savedSearchType={SavedSearchType.ISSUE}
         onGetTagValues={this.getTagValues}
         onGetTagValues={this.getTagValues}
         defaultSearchItems={this.state.defaultSearchItems}
         defaultSearchItems={this.state.defaultSearchItems}
@@ -143,6 +142,7 @@ class IssueListSearchBar extends Component<Props, State> {
           makeSaveSearchAction({sort}),
           makeSaveSearchAction({sort}),
           makeSearchBuilderAction({onSidebarToggle}),
           makeSearchBuilderAction({onSidebarToggle}),
         ]}
         ]}
+        maxMenuHeight={500}
         {...props}
         {...props}
       />
       />
     );
     );

+ 1 - 1
static/app/views/projectDetail/projectFilters.tsx

@@ -36,7 +36,6 @@ function ProjectFilters({query, relativeDateOptions, tagValueLoader, onSearch}:
           searchSource="project_filters"
           searchSource="project_filters"
           query={query}
           query={query}
           placeholder={t('Search by release version, build, package, or stage')}
           placeholder={t('Search by release version, build, package, or stage')}
-          maxSearchItems={5}
           hasRecentSearches={false}
           hasRecentSearches={false}
           supportedTags={{
           supportedTags={{
             ...SEMVER_TAGS,
             ...SEMVER_TAGS,
@@ -45,6 +44,7 @@ function ProjectFilters({query, relativeDateOptions, tagValueLoader, onSearch}:
               name: 'release',
               name: 'release',
             },
             },
           }}
           }}
+          maxMenuHeight={500}
           onSearch={onSearch}
           onSearch={onSearch}
           onGetTagValues={getTagValues}
           onGetTagValues={getTagValues}
         />
         />

+ 1 - 1
static/app/views/releases/list/index.tsx

@@ -544,7 +544,6 @@ class ReleasesList extends AsyncView<Props, State> {
                   searchSource="releases"
                   searchSource="releases"
                   query={this.getQuery()}
                   query={this.getQuery()}
                   placeholder={t('Search by version, build, package, or stage')}
                   placeholder={t('Search by version, build, package, or stage')}
-                  maxSearchItems={5}
                   hasRecentSearches={false}
                   hasRecentSearches={false}
                   supportedTags={{
                   supportedTags={{
                     ...SEMVER_TAGS,
                     ...SEMVER_TAGS,
@@ -553,6 +552,7 @@ class ReleasesList extends AsyncView<Props, State> {
                       name: 'release',
                       name: 'release',
                     },
                     },
                   }}
                   }}
+                  maxMenuHeight={500}
                   supportedTagType={ItemType.PROPERTY}
                   supportedTagType={ItemType.PROPERTY}
                   onSearch={this.handleSearch}
                   onSearch={this.handleSearch}
                   onGetTagValues={this.getTagValues}
                   onGetTagValues={this.getTagValues}