[BUG] No space between the detector details and the rule panel opense…

…arch-project#522 [BUG] A rule flyout without references have an empty link opensearch-project#521 [FEATURE] Update header size to use euiTitle--small opensearch-project#520 Signed-off-by: Jovan Cvetkovic <jovanca.cvetkovic@gmail.com>
jovancacvetkovic · Apr 17, 2023 · 2a11ebe · 2a11ebe
1 parent fdcc9c5
commit 2a11ebe
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 5 deletions.
diff --git a/...ages/CreateDetector/components/ConfigureFieldMapping/containers/ConfigureFieldMapping.tsx b/...ages/CreateDetector/components/ConfigureFieldMapping/containers/ConfigureFieldMapping.tsx
@@ -253,7 +253,7 @@ export default class ConfigureFieldMapping extends Component<
             <EuiAccordion
               buttonContent={
                 <div data-test-subj="mapped-fields-btn">
-                  <EuiTitle>
+                  <EuiTitle size={'s'}>
                     <h6>{`Automatically mapped fields (${mappedRuleFields.length})`}</h6>
                   </EuiTitle>
                 </div>
@@ -303,7 +303,7 @@ export default class ConfigureFieldMapping extends Component<
 
                   <EuiSpacer size={'m'} />
 
-                  <EuiTitle>
+                  <EuiTitle size={'s'}>
                     <h6>Pending field mappings</h6>
                   </EuiTitle>
                   <FieldMappingsTable<MappingViewType.Edit>

diff --git a/...ges/CreateDetector/components/DefineDetector/components/DetectionRules/DetectionRules.tsx b/...ges/CreateDetector/components/DefineDetector/components/DetectionRules/DetectionRules.tsx
@@ -92,7 +92,7 @@ export const DetectionRules: React.FC<DetectionRulesProps> = ({
       <EuiAccordion
         buttonContent={
           <div data-test-subj="detection-rules-btn">
-            <EuiTitle>
+            <EuiTitle size={'s'}>
               <h4>{`Detection rules (${enabledRulesCount} selected)`}</h4>
             </EuiTitle>
             <EuiText size="s" color="subdued">

diff --git a/public/pages/Detectors/containers/DetectorDetailsView/DetectorDetailsView.tsx b/public/pages/Detectors/containers/DetectorDetailsView/DetectorDetailsView.tsx
@@ -4,6 +4,7 @@
  */
 
 import React from 'react';
+import { EuiSpacer } from '@elastic/eui';
 import { DetectorBasicDetailsView } from '../../components/DetectorBasicDetailsView/DetectorBasicDetailsView';
 import { DetectorRulesView } from '../../components/DetectorRulesView/DetectorRulesView';
 import { RuleItem } from '../../../CreateDetector/components/DefineDetector/components/DetectionRules/types/interfaces';
@@ -64,7 +65,12 @@ export class DetectorDetailsView extends React.Component<
           {rulesCanFold ? detectorRules : null}
         </DetectorBasicDetailsView>
 
-        {rulesCanFold ? null : detectorRules}
+        {rulesCanFold ? null : (
+          <>
+            <EuiSpacer size={'m'} />
+            {detectorRules}
+          </>
+        )}
       </>
     );
   }

diff --git a/public/pages/Rules/components/RuleEditor/RuleEditorFormModel.ts b/public/pages/Rules/components/RuleEditor/RuleEditorFormModel.ts
@@ -28,7 +28,7 @@ export const ruleEditorStateDefaultValue: RuleEditorFormModel = {
   description: '',
   status: '',
   author: '',
-  references: [''],
+  references: [],
   tags: [],
   detection: '',
   level: '',