chore(topsql): hide limit select for op (#1689)

pingcap · Jun 12, 2024 · 91f6c28 · 91f6c28
1 parent 3318058
commit 91f6c28
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 13 deletions.
diff --git a/ui/packages/tidb-dashboard-for-op/src/apps/TopSQL/context.ts b/ui/packages/tidb-dashboard-for-op/src/apps/TopSQL/context.ts
@@ -48,6 +48,7 @@ export const ctx: ITopSQLContext = {
   ds,
   cfg: {
     checkNgm: true,
-    showSetting: true
+    showSetting: true,
+    showLimit: true
   }
 }
diff --git a/ui/packages/tidb-dashboard-lib/src/apps/TopSQL/context/index.ts b/ui/packages/tidb-dashboard-lib/src/apps/TopSQL/context/index.ts
@@ -52,6 +52,7 @@ export interface ITopSQLConfig {
   userName?: string
 
   showSearchInStatements?: boolean
+  showLimit?: boolean
 }
 
 export interface ITopSQLContext {

diff --git a/ui/packages/tidb-dashboard-lib/src/apps/TopSQL/pages/List/List.tsx b/ui/packages/tidb-dashboard-lib/src/apps/TopSQL/pages/List/List.tsx
@@ -217,18 +217,20 @@ export function TopSQLList() {
                 }
                 disabled={isLoading}
               />
-              <Select
-                style={{ width: 150 }}
-                value={limit}
-                onChange={setLimit}
-                data-e2e="limit_select"
-              >
-                {LIMITS.map((item) => (
-                  <Option value={item} key={item} data-e2e="limit_option">
-                    Limit {item}
-                  </Option>
-                ))}
-              </Select>
+              {ctx?.cfg.showLimit && (
+                <Select
+                  style={{ width: 150 }}
+                  value={limit}
+                  onChange={setLimit}
+                  data-e2e="limit_select"
+                >
+                  {LIMITS.map((item) => (
+                    <Option value={item} key={item} data-e2e="limit_option">
+                      Limit {item}
+                    </Option>
+                  ))}
+                </Select>
+              )}
 
               <AutoRefreshButton
                 defaultValue={ctx?.cfg.autoRefresh === false ? 0 : undefined}