[SIEM] Detection Engine Create Rule Design Review #1#54442
Conversation
|
Pinging @elastic/siem (Team:SIEM) |
MichaelMarcialis
left a comment
There was a problem hiding this comment.
This is looking great! Thanks so much for the fast fixes, @patrykkopycinski! Leaving a few small comments below.
-
Thanks for correcting the alignment/indentation of the form fields. Unfortunately, it looks like this change also indented in the
hrat the bottom of each step panel (above the "Continue" buttons). Is there a way that the bottomhrelements can not be indented and instead match the alignment and width of the tophrs, as shown in the designs? -
It look like with these changes, the completed step summaries are now laid out in a single column. Is there any way we can restore the completed step summaries to being in a two column layout (even if "Description" field can’t go the full width to match the designs)?
...egacy/plugins/siem/public/pages/detection_engine/rules/components/step_about_rule/schema.tsx
Outdated
Show resolved
Hide resolved
.../plugins/siem/public/pages/detection_engine/rules/components/step_about_rule/translations.ts
Outdated
Show resolved
Hide resolved
...egacy/plugins/siem/public/pages/detection_engine/rules/components/step_define_rule/index.tsx
Outdated
Show resolved
Hide resolved
…tion-engine-design-review-1 # Conflicts: # x-pack/legacy/plugins/siem/public/pages/detection_engine/rules/all/columns.tsx # x-pack/legacy/plugins/siem/public/pages/detection_engine/rules/create/index.tsx
…tion-engine-design-review-1
|
There was a problem hiding this comment.
This looks great, @patrykkopycinski. I only noticed one super small thing, but marking this as approved for after. Comment below. Thanks!
It appears that the "Optional" text for "Additional look-back" field is the incorrect color (should match the rest). Also when the field is focused, it turns blue (like the label). That's not a pattern that happens in the fields above. If we can lose the focus color on the "Optional" text here, that would be great.
|
Thank you @MichaelMarcialis 👍 |
|
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
* upstream/master: (26 commits) Take page offset into account too (elastic#54567) [APM] Support error.{log,exception}.stacktrace.classname (elastic#54577) Np migration tsvb route validation (elastic#51850) [ML] MML calculator enhancements for multi-metric job wizard (elastic#54573) [SIEM] Fix Inspect query 'request timestamp' value changes when curso… (elastic#54223) Fix chromeless NP apps not using full page width (elastic#54550) Remove extraneous public import to prevent failing Kibana startup (elastic#54676) [Uptime] Temporarily skip flakey tests (elastic#54675) Skip failing uptime tests Create UI for alerting and actions plugin (elastic#48959) [dev/build/sass] build stylesheets for disabled plugins too (elastic#54654) [SIEM] Use bulk actions API when updating or deleting rules (elastic#54521) Support "Deprecated" label in advanced settings (elastic#54539) [Maps] add text halo color and width style properties (elastic#53827) Service Map Data API at Runtime (elastic#54027) [SIEM] Detection Engine Create Rule Design Review #1 (elastic#54442) Skip flaky test [Canvas] Enable Embeddable maps (elastic#53971) [SIEM][Detection Engine] Increases the number or rules you can view on a single page (elastic#54628) uiSettings - use validation field for image field maxSize (elastic#54522) ...











Summary
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.For maintainers