From cfc9bbcdbf4a21c96093afe008d45b3156e9a8d0 Mon Sep 17 00:00:00 2001
From: CaIon <i@caion.me>
Date: Thu, 18 Jun 2026 21:29:55 +0800
Subject: [PATCH] feat(logs): improve layout

---
 .../components/logs-filter-toolbar.tsx        | 74 +++++++++++--------
 1 file changed, 43 insertions(+), 31 deletions(-)

diff --git a/web/default/src/features/usage-logs/components/logs-filter-toolbar.tsx b/web/default/src/features/usage-logs/components/logs-filter-toolbar.tsx
index 63f9b207..e02824c5 100644
--- a/web/default/src/features/usage-logs/components/logs-filter-toolbar.tsx
+++ b/web/default/src/features/usage-logs/components/logs-filter-toolbar.tsx
@@ -103,6 +103,34 @@ export function LogsFilterToolbar<TData>(props: LogsFilterToolbarProps<TData>) {
     setMobileFiltersOpen(false)
   }
 
+  const advancedToggle = hasAdvancedFilters ? (
+    <Button
+      type='button'
+      variant='ghost'
+      onClick={() => setAdvancedOpen((open) => !open)}
+      aria-expanded={advancedOpen}
+      className={cn(
+        'text-muted-foreground hover:text-foreground gap-1 px-2',
+        props.hasAdvancedActiveFilters &&
+          !advancedOpen &&
+          'text-primary hover:text-primary'
+      )}
+    >
+      {advancedOpen ? t('Collapse') : t('Expand')}
+      {activeAdvancedCount > 0 && (
+        <Badge className='ml-0.5 size-5 justify-center p-0 text-[10px]'>
+          {activeAdvancedCount}
+        </Badge>
+      )}
+      <ChevronDown
+        className={cn(
+          'size-3.5 transition-transform duration-200',
+          advancedOpen && 'rotate-180'
+        )}
+      />
+    </Button>
+  ) : null
+
   if (isMobile && props.mobilePinnedFilters != null) {
     return (
       <Drawer open={mobileFiltersOpen} onOpenChange={setMobileFiltersOpen}>
@@ -192,42 +220,26 @@ export function LogsFilterToolbar<TData>(props: LogsFilterToolbarProps<TData>) {
         props.className
       )}
     >
-      <div className='grid grid-cols-1 gap-2 sm:grid-cols-[repeat(auto-fit,minmax(10rem,1fr))]'>
-        {props.primaryFilters}
-        {advancedOpen && props.advancedFilters}
+      <div className='flex flex-wrap items-start gap-2'>
+        <div className='grid min-w-0 flex-1 grid-cols-1 gap-2 sm:grid-cols-[repeat(auto-fit,minmax(10rem,1fr))]'>
+          {props.primaryFilters}
+        </div>
+        {advancedToggle && (
+          <div className='flex shrink-0 items-center justify-end'>
+            {advancedToggle}
+          </div>
+        )}
       </div>
 
+      {advancedOpen && props.advancedFilters && (
+        <div className='mt-2 grid grid-cols-1 gap-2 sm:grid-cols-[repeat(auto-fit,minmax(10rem,1fr))]'>
+          {props.advancedFilters}
+        </div>
+      )}
+
       <div className='mt-2 flex flex-wrap items-center gap-2'>
         {props.stats}
         <div className='ms-auto flex flex-wrap items-center justify-end gap-1.5 sm:gap-2'>
-          {hasAdvancedFilters && (
-            <Button
-              type='button'
-              variant='ghost'
-              onClick={() => setAdvancedOpen((open) => !open)}
-              aria-expanded={advancedOpen}
-              className={cn(
-                'text-muted-foreground hover:text-foreground gap-1 px-2',
-                props.hasAdvancedActiveFilters &&
-                  !advancedOpen &&
-                  'text-primary hover:text-primary'
-              )}
-            >
-              {advancedOpen ? t('Collapse') : t('Expand')}
-              {activeAdvancedCount > 0 && (
-                <Badge className='ml-0.5 size-5 justify-center p-0 text-[10px]'>
-                  {activeAdvancedCount}
-                </Badge>
-              )}
-              <ChevronDown
-                className={cn(
-                  'size-3.5 transition-transform duration-200',
-                  advancedOpen && 'rotate-180'
-                )}
-              />
-            </Button>
-          )}
-
           <Button
             type='button'
             variant='outline'