Search: Fix not being able to clear sort value (#62557)

* user essentials mob! 🔱

lastFile:public/app/features/search/state/SearchStateManager.ts

* user essentials mob! 🔱

lastFile:public/app/features/search/page/components/ActionRow.tsx

* user essentials mob! 🔱

* remove searchSort state from localStorage when cleared

---------

Co-authored-by: Ashley Harrison <ashley.harrison@grafana.com>
Co-authored-by: Joao Silva <joao.silva@grafana.com>
pull/62879/head
Josh Hunt 2 years ago committed by GitHub
parent 12d192d80e
commit f8809eef59
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      public/app/features/search/page/components/ActionRow.tsx
  2. 6
      public/app/features/search/state/SearchStateManager.ts

@ -106,10 +106,10 @@ export const ActionRow: FC<Props> = ({
/>
)}
<SortPicker
onChange={(change) => onSortChange(change.value)}
onChange={(change) => onSortChange(change?.value)}
value={state.sort}
getSortOptions={getSortOptions}
placeholder={sortPlaceholder}
placeholder={sortPlaceholder || 'Sort'}
isClearable
/>
</HorizontalGroup>

@ -115,6 +115,8 @@ export class SearchStateManager extends StateManagerBase<SearchState> {
onSortChange = (sort: string | undefined) => {
if (sort) {
localStorage.setItem(SEARCH_SELECTED_SORT, sort);
} else {
localStorage.removeItem(SEARCH_SELECTED_SORT);
}
if (this.state.layout === SearchLayout.Folders) {
@ -171,10 +173,6 @@ export class SearchStateManager extends StateManagerBase<SearchState> {
q.kind = ['dashboard', 'folder']; // skip panels
}
if (q.query === '*' && !q.sort?.length) {
q.sort = 'name_sort';
}
return q;
}

Loading…
Cancel
Save