From fe757f69c57bec89be8e17a9c8022e0334c9e0e5 Mon Sep 17 00:00:00 2001 From: Khushi Jain Date: Tue, 2 Jan 2024 19:36:23 +0530 Subject: [PATCH] Forms: Remove gf-form from opentsdb/components (#79830) * Forms: Remove gf-form from opentsdb/components * filter and tag * rate * Update public/app/plugins/datasource/opentsdb/components/DownSample.tsx Co-authored-by: Alex Khomenko * Update public/app/plugins/datasource/opentsdb/components/FilterSection.tsx Co-authored-by: Alex Khomenko * Update public/app/plugins/datasource/opentsdb/components/MetricSection.tsx Co-authored-by: Alex Khomenko * Update public/app/plugins/datasource/opentsdb/components/RateSection.tsx Co-authored-by: Alex Khomenko * Update public/app/plugins/datasource/opentsdb/components/TagSection.tsx Co-authored-by: Alex Khomenko * prettier * more changes --------- Co-authored-by: Alex Khomenko --- .betterer.results | 73 +----------- .../opentsdb/components/DownSample.tsx | 32 +++-- .../opentsdb/components/FilterSection.tsx | 103 ++++++++-------- .../opentsdb/components/MetricSection.tsx | 26 ++-- .../components/OpenTsdbQueryEditor.tsx | 9 +- .../opentsdb/components/RateSection.tsx | 51 ++++---- .../opentsdb/components/TagSection.tsx | 112 ++++++++++-------- 7 files changed, 166 insertions(+), 240 deletions(-) diff --git a/.betterer.results b/.betterer.results index 26ae030adbb..8cb345bb383 100644 --- a/.betterer.results +++ b/.betterer.results @@ -5550,8 +5550,7 @@ exports[`better eslint`] = { ], "public/app/plugins/datasource/opentsdb/components/OpenTsdbQueryEditor.tsx:5381": [ [0, 0, 0, "Styles should be written using objects.", "0"], - [0, 0, 0, "Styles should be written using objects.", "1"], - [0, 0, 0, "Styles should be written using objects.", "2"] + [0, 0, 0, "Styles should be written using objects.", "1"] ], "public/app/plugins/datasource/opentsdb/datasource.d.ts:5381": [ [0, 0, 0, "Unexpected any. Specify a different type.", "0"] @@ -7405,76 +7404,6 @@ exports[`no gf-form usage`] = { [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"] ], - "public/app/plugins/datasource/opentsdb/components/DownSample.tsx:5381": [ - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"] - ], - "public/app/plugins/datasource/opentsdb/components/FilterSection.tsx:5381": [ - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"] - ], - "public/app/plugins/datasource/opentsdb/components/MetricSection.tsx:5381": [ - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"] - ], - "public/app/plugins/datasource/opentsdb/components/RateSection.tsx:5381": [ - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"] - ], - "public/app/plugins/datasource/opentsdb/components/TagSection.tsx:5381": [ - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], - [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"] - ], "public/app/plugins/datasource/prometheus/components/PromExploreExtraField.tsx:5381": [ [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], [0, 0, 0, "gf-form usage has been deprecated. Use a component from @grafana/ui or custom CSS instead.", "5381"], diff --git a/public/app/plugins/datasource/opentsdb/components/DownSample.tsx b/public/app/plugins/datasource/opentsdb/components/DownSample.tsx index ab022d66f48..74911259d50 100644 --- a/public/app/plugins/datasource/opentsdb/components/DownSample.tsx +++ b/public/app/plugins/datasource/opentsdb/components/DownSample.tsx @@ -1,12 +1,10 @@ import React from 'react'; import { toOption } from '@grafana/data'; -import { InlineLabel, Select, Input, InlineFormLabel, InlineSwitch } from '@grafana/ui'; +import { InlineLabel, Select, Input, InlineFormLabel, InlineSwitch, Stack } from '@grafana/ui'; import { OpenTsdbQuery } from '../types'; -import { paddingRightClass } from './styles'; - export interface DownSampleProps { query: OpenTsdbQuery; onChange: (query: OpenTsdbQuery) => void; @@ -21,8 +19,8 @@ export function DownSample({ query, onChange, onRunQuery, aggregators, fillPolic const fillPolicyOptions = fillPolicies.map((value: string) => toOption(value)); return ( -
-
+ + onRunQuery()} /> -
-
+ + Aggregator -
+ )} -
+ Disable downsampling -
-
-
-
-
+ + + + + ); } diff --git a/public/app/plugins/datasource/opentsdb/components/FilterSection.tsx b/public/app/plugins/datasource/opentsdb/components/FilterSection.tsx index c947b0017db..86c1d0a917f 100644 --- a/public/app/plugins/datasource/opentsdb/components/FilterSection.tsx +++ b/public/app/plugins/datasource/opentsdb/components/FilterSection.tsx @@ -12,6 +12,7 @@ import { clearButtonStyles, useStyles2, AsyncSelect, + Stack, } from '@grafana/ui'; import { OpenTsdbFilter, OpenTsdbQuery } from '../types'; @@ -118,46 +119,45 @@ export function FilterSection({ const tagValueSearch = debounce((query: string) => suggestTagValues(query), 350); return ( -
-
- Filters does not work with tags, either of the two will work but not both.
} - > - Filters - - {query.filters && - query.filters.map((fil: OpenTsdbFilter, idx: number) => { - return ( - - {fil.tagk} = {fil.type}({fil.filter}), groupBy = {'' + fil.groupBy} - - - - ); - })} - {!addFilterMode && ( -
} + > + Filters + + {query.filters && + query.filters.map((fil: OpenTsdbFilter, idx: number) => { + return ( + + {fil.tagk} = {fil.type}({fil.filter}), groupBy = {'' + fil.groupBy} + + + + ); + })} + {!addFilterMode && ( + + - )} - + + )} {addFilterMode && ( -
-
+ + -
+ -
+ -
+ Group by @@ -221,27 +220,27 @@ export function FilterSection({ updCurFilterGroupBy(!curFilterGroupBy); }} /> -
+ {errors && ( -
+ -
+ )} -
+ -
-
-
+ + + )} -
-
-
- + + + + ); } diff --git a/public/app/plugins/datasource/opentsdb/components/MetricSection.tsx b/public/app/plugins/datasource/opentsdb/components/MetricSection.tsx index edb3adad010..a76ff1b4422 100644 --- a/public/app/plugins/datasource/opentsdb/components/MetricSection.tsx +++ b/public/app/plugins/datasource/opentsdb/components/MetricSection.tsx @@ -2,7 +2,7 @@ import debounce from 'debounce-promise'; import React from 'react'; import { SelectableValue, toOption } from '@grafana/data'; -import { Select, Input, InlineFormLabel, AsyncSelect } from '@grafana/ui'; +import { Select, Input, InlineFormLabel, AsyncSelect, Stack, InlineLabel } from '@grafana/ui'; import { OpenTsdbQuery } from '../types'; @@ -19,8 +19,8 @@ export function MetricSection({ query, onChange, onRunQuery, suggestMetrics, agg const metricSearch = debounce((query: string) => suggestMetrics(query), 350); return ( -
-
+ + Metric @@ -28,7 +28,6 @@ export function MetricSection({ query, onChange, onRunQuery, suggestMetrics, agg -
-
+ + Aggregator -
+ -
+ -
+ -
+ {errors && ( -
+ -
+ )} -
+ -
-
-
+ + + )} -
-
-
- + + + + ); }