Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Security Solution] Suggestion: Alerting application needs to used fixed layouts for proper scrolling / virtualization #120291

Open
Tracked by #120277
snide opened this issue Dec 2, 2021 · 3 comments
Assignees
Labels
Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc.

Comments

@snide
Copy link
Contributor

snide commented Dec 2, 2021

The current Alerting page layout should likely utilize a fixed layout similar to Discover to better expose affordability around scrolling, pagination and screen real estate.

Take a look at discover. It has the following benefits.

  • The horizontal scroll if needed is always available
  • The controls for the grid remain in the same position
  • The controls for pagination remain in the same position and are known
  • The charts are vertically minimal and can be hidden
  • Real estate is focused on the content of the results
  • Because a height is set, the grid can be properly virtualized, making it MUCH more performant

Now take a look at the alerting table view

image

image

@snide snide added bug Fixes for quality problems that affect the customer experience triage_needed Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. labels Dec 2, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@Max-Iskra
Copy link

Max-Iskra commented Feb 22, 2022

Figma mockup/prototype: https://ela.st/proper-scrolling

Proper-Scrolling.mp4
  • The table scrolls if needed are always available

Proper-scroll

Exploration

  • We can use the button without a dropdown for better discoverability since there's only one action for hiding/showing charts

Hide

cc @snide @paulewing

@Max-Iskra
Copy link

Updated view without big title based on issue #120287

Chart Shown

Alerts Header

Chart Hided

Alerts Header-collapsed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc.
Projects
None yet
Development

No branches or pull requests

3 participants