Skip to content

Commit 7e3b651

Browse files
fix(alerts): Fix EAP alert filter bar to behave more like explore (#81946)
Updates the search bar in the EAP Alert edit page to behave more like explore: - User must explicitly save changes on the search (hitting enter) for updates to take effect. Same as explore. - Removed onBlur handler This also fixes an issue where the chart visualization updates, but saving the alert rule doesn't actually save the changed filter because it was not actually committed
1 parent 2d1ca2a commit 7e3b651

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

static/app/views/alerts/rules/metric/ruleConditionsForm.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -781,16 +781,12 @@ class RuleConditionsForm extends PureComponent<Props, State> {
781781
<EAPSpanSearchQueryBuilder
782782
numberTags={tags?.number ?? {}}
783783
stringTags={tags?.string ?? {}}
784-
initialQuery={initialData?.query ?? ''}
784+
initialQuery={value ?? ''}
785785
searchSource="alerts"
786786
onSearch={(query, {parsedQuery}) => {
787787
onFilterSearch(query, parsedQuery);
788788
onChange(query, {});
789789
}}
790-
onBlur={(query, {parsedQuery}) => {
791-
onFilterSearch(query, parsedQuery);
792-
onBlur(query);
793-
}}
794790
supportedAggregates={ALLOWED_EXPLORE_VISUALIZE_AGGREGATES}
795791
projects={[parseInt(project.id, 10)]}
796792
/>

0 commit comments

Comments
 (0)