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] No Horizontal Scroll bar under Alert Full Screen View lead to all field not show #109427

Closed
ghost opened this issue Aug 20, 2021 · 11 comments · Fixed by #111492
Assignees
Labels
bug Fixes for quality problems that affect the customer experience impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. QA:Validated Issue has been validated by QA Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting Security Solution Threat Hunting Team Theme: rac label obsolete v7.15.0

Comments

@ghost
Copy link

ghost commented Aug 20, 2021

Describe the bug
No Horizontal Scroll bar under Alert Full Screen View lead to all field not show

Build Details

Version:7.15.0
Commit:d791226d9385122f33f4a5ca38fa5369012fbec3
Build:43636

Browsers
all

Precondition

  • Install the Endpoint Security on the kibana

Steps to Reproduce

  1. Generate few Alerts .
  2. Click on Filed icon and add more field let more that 20.
  3. Click on Full Screen.
  4. Observed that all the field information not show under the Alert Full Screen , user need to lowed the browser zoom and on decreasing browser zoom data becomes not readable.

Actual Result
No Horizontal Scroll bar under Alert Full Screen View due to which all field not show

Expected Result
No Horizontal Scroll bar under Alert Full Screen View lead to all field not show

Screen-Cast

full-screen-fields.mp4

logs
N/A

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

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

@manishgupta-qasource
Copy link

Reviewed & assigned to @MadameSheema

@manishgupta-qasource manishgupta-qasource added the impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. label Aug 20, 2021
@MadameSheema MadameSheema added Team:Threat Hunting Security Solution Threat Hunting Team triage_needed labels Aug 20, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@MadameSheema MadameSheema removed their assignment Aug 20, 2021
@MadameSheema MadameSheema added the Theme: rac label obsolete label Aug 25, 2021
@machadoum machadoum added the fixed label Sep 1, 2021
@machadoum machadoum removed their assignment Sep 1, 2021
@machadoum
Copy link
Member

I am not able to reproduce it on master. I tested it on chrome 92.0.4515.159, safari 14.1.1, and Firefox 91.0.2.

@ghost
Copy link
Author

ghost commented Sep 1, 2021

HI @machadoum

we have retested and found it still occuring on our hardware machines.

The Horizontal scroll bar is not present in the full screen mode . Make to sure add more field on alert list let say 16 . That way you can check you were not able to read all 16 field data due to no horizontal bar , plus with browser zoom out field start showing but becomes unreadable.

we will re-test the issue on BC4

Please find below all details:

Build Details:

Version: 7.15.0 BC3
Commit:6f7562b1906dcfad65809da8fdec15df353d0252
Build:43818

Windows Browsers
Firefox: 91.0.2
Chrome: 93.0.4577.63

Screen-Resolution: 1366 X 768

Screen-Cast:

Firefox:
https://user-images.githubusercontent.com/59917825/131652524-51c8cbd1-722f-4950-b395-da916d090a04.mp4
Chrome :
image

@machadoum
Copy link
Member

You are absolutely right! It only showed up to me because I have a magic mouse, so I can scroll horizontally. I am taking a look at it.

@machadoum
Copy link
Member

I was able to isolate the issue and trace the origin back to EuiDataGrid. So, I created this elastic/eui#5131 issue for the Eui team.

@MadameSheema MadameSheema removed the fixed label Sep 2, 2021
@MadameSheema MadameSheema removed their assignment Sep 2, 2021
@machadoum
Copy link
Member

machadoum commented Sep 2, 2021

Another related issue:
When the alert table is full screen, the timeline flyout bar covers the alerts table footer completely. So the user can't change the page any longer. @mdefazio What should happen here? Should we hide the timeline bar?

Screenshot 2021-09-02 at 14 57 52

@MadameSheema
Copy link
Member

@karanbirsingh-qasource please validate this issue on 7.15BC6? Thanks :)

@mdefazio
Copy link
Contributor

mdefazio commented Sep 9, 2021

When the alert table is full screen, the timeline flyout bar covers the alerts table footer completely. So the user can't change the page any longer.

I think the timeline bar should not be shown. We are hiding the search bar and so I feel this should follow the same logic. We also no longer have drag-n-drop, and when adding a field to the timeline, we show a toast (correct?).

Is this fixed in the provided PR that closes this? My apologies for missing this mention until now.

cc/ @adamwdraper Just FYI as this is Security specific

@ghost
Copy link
Author

ghost commented Sep 15, 2021

Hi @MadameSheema

As per the update from @mdefazio about the expected behavior for not showing the horizontal scroll and the other issue of timeline bar covering row per page and next page is now fixed, so closing this issue.

Build Details:

Version: 7.15.0 BC6
Commit:ab43f574e8ea01f5093fe92b2f51c4b686eb6e63
Build:44026

Snap-Shoot:
image

Hence we are closing this issue and adding "QA:Validated" label to it.

thanks !!

@ghost ghost added the QA:Validated Issue has been validated by QA label Sep 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. QA:Validated Issue has been validated by QA Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting Security Solution Threat Hunting Team Theme: rac label obsolete v7.15.0
Projects
None yet
6 participants