Move buttons and make active the default

elastic · May 25, 2021 · 3b17d0d · 3b17d0d
1 parent ba2e5ae
commit 3b17d0d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/x-pack/plugins/observability/public/pages/alerts/index.tsx b/x-pack/plugins/observability/public/pages/alerts/index.tsx
@@ -60,7 +60,7 @@ export function AlertsPage({ routeParams }: AlertsPageProps) {
   const { prepend } = core.http.basePath;
   const history = useHistory();
   const {
-    query: { rangeFrom = 'now-15m', rangeTo = 'now', kuery = '', status = 'all' },
+    query: { rangeFrom = 'now-15m', rangeTo = 'now', kuery = '', status = 'active' },
   } = routeParams;
 
   // In a future milestone we'll have a page dedicated to rule management in

diff --git a/x-pack/plugins/observability/public/pages/alerts/status_filter.tsx b/x-pack/plugins/observability/public/pages/alerts/status_filter.tsx
@@ -18,16 +18,6 @@ export interface StatusFilterProps {
 export function StatusFilter({ status = 'open', onChange }: StatusFilterProps) {
   return (
     <EuiFilterGroup>
-      <EuiFilterButton
-        data-test-subj="StatusFilter all button"
-        hasActiveFilters={status === 'all'}
-        onClick={() => onChange('all')}
-        withNext={true}
-      >
-        {i18n.translate('xpack.observability.alerts.statusFilter.allButtonLabel', {
-          defaultMessage: 'All',
-        })}
-      </EuiFilterButton>
       <EuiFilterButton
         data-test-subj="StatusFilter open button"
         hasActiveFilters={status === 'open'}
@@ -42,11 +32,21 @@ export function StatusFilter({ status = 'open', onChange }: StatusFilterProps) {
         data-test-subj="StatusFilter closed button"
         hasActiveFilters={status === 'closed'}
         onClick={() => onChange('closed')}
+        withNext={true}
       >
         {i18n.translate('xpack.observability.alerts.statusFilter.recoveredButtonLabel', {
           defaultMessage: 'Recovered',
         })}
       </EuiFilterButton>
+      <EuiFilterButton
+        data-test-subj="StatusFilter all button"
+        hasActiveFilters={status === 'all'}
+        onClick={() => onChange('all')}
+      >
+        {i18n.translate('xpack.observability.alerts.statusFilter.allButtonLabel', {
+          defaultMessage: 'All',
+        })}
+      </EuiFilterButton>
     </EuiFilterGroup>
   );
 }