Skip to content

[8.1] [Security Solution][Detections] fixes UX issue with discoverability of new features on the Rules page (#124343)#124992

Merged
kibanamachine merged 2 commits intoelastic:8.1from
kibanamachine:backport/8.1/pr-124343
Feb 9, 2022
Merged

[8.1] [Security Solution][Detections] fixes UX issue with discoverability of new features on the Rules page (#124343)#124992
kibanamachine merged 2 commits intoelastic:8.1from
kibanamachine:backport/8.1/pr-124343

Conversation

@kibanamachine
Copy link
Contributor

…f new features on the Rules page (elastic#124343)

- Added tour with 2  steps
   - Sorting/filtering for in memory table
   - New Bulk Edit actions
- Created context with tour management, which wraps rules table
- Results of user tour's journey is saved in localStorage
- Disabled tour for Cypress tests

**Note:** Text in second step was changed to "You can now bulk update index patterns and tags for multiple custom rules at once." It's not reflected on screenshots. On screenshots and video present older text: "You can now bulk update index patterns and tags for multiple rules at once.", where word `custom` is absent

## UI

### First step
<img width="1287" alt="Screenshot 2022-02-03 at 16 54 20" src="https://user-images.githubusercontent.com/92328789/152391079-57a12b97-a273-4852-95e6-222680d09464.png">

### Second step
<img width="1281" alt="Screenshot 2022-02-03 at 16 54 29" src="https://user-images.githubusercontent.com/92328789/152391090-87cd436a-eb74-4358-b699-53c3e79fdb48.png">

Eui Tour: https://elastic.github.io/eui/#/display/tour
## Screen recording

https://user-images.githubusercontent.com/92328789/152391296-0b03b299-270a-4d96-9adf-d98bc6405b4f.mov

### Checklist

Delete any items that are not applicable to this PR.

- [x] Any text added follows [EUI's writing guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/main/packages/kbn-i18n/README.md)
- [x] Any UI touched in this PR is usable by keyboard only (learn more about [keyboard accessibility](https://webaim.org/techniques/keyboard/))
- [x] Any UI touched in this PR does not create any new axe failures (run axe in browser: [FF](https://addons.mozilla.org/en-US/firefox/addon/axe-devtools/), [Chrome](https://chrome.google.com/webstore/detail/axe-web-accessibility-tes/lhdoppojpmngadmnindnejefpokejbdd?hl=en-US))

(cherry picked from commit 6f71ffc)
@vitaliidm
Copy link
Contributor

@elasticmachine merge upstream

@kibanamachine kibanamachine merged commit 347107a into elastic:8.1 Feb 9, 2022
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 2868 2870 +2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 4.7MB 4.7MB +3.2KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
securitySolution 246.0KB 246.1KB +99.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @vitaliidm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants