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] The user is able to select multiple values from the alerts detail flyout. Also the modal does not stick to the UI, it moves on scrolling the UI #108869

Closed
muskangulati-qasource opened this issue Aug 17, 2021 · 11 comments
Assignees
Labels
bug Fixes for quality problems that affect the customer experience impact:low Addressing this issue will have a low 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 v8.10.0

Comments

@muskangulati-qasource
Copy link

Description:
The user is able to select multiple values from the alerts detail flyout. Also the modal does not stick to the UI, it moves on scrolling the UI

Build Details:

Version : 8.0.0-SNAPSHOT
Commit:9aa1705cd9546ed43af59d790c58e687f5716ca1
Build:45368

Browser Details:
All

Preconditions:

  1. Kibana user should be logged in.
  2. Alert should be generated

Steps to Reproduce:

  1. Navigate to the alerts tab.
  2. Click on '>' icon
  3. Observe the user can click on multiple entries.

Impacted Test case:
N/A

Actual Result:
The user is able to select multiple values from the alerts detail flyout. Also the modal does not stick to the UI, it moves on scrolling the UI

Expected Result:
Multiple values should not be selectable

What's working:
N/A

What's not working:
N/A

Screenshots:
MultipleValuesAreSelectable

Logs:
N/A

@muskangulati-qasource muskangulati-qasource 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 17, 2021
@elasticmachine
Copy link
Contributor

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

@muskangulati-qasource muskangulati-qasource added v7.15.0 impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. labels Aug 17, 2021
@manishgupta-qasource
Copy link

Reviewed & assigned to @MadameSheema

@MadameSheema MadameSheema added Team:Threat Hunting Security Solution Threat Hunting Team and removed v7.15.0 labels Aug 17, 2021
@elasticmachine
Copy link
Contributor

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

@MadameSheema MadameSheema removed their assignment Aug 17, 2021
@MadameSheema
Copy link
Member

@muskangulati-qasource can you please validate if this issue is happening on 7.x branch? thanks

@angorayc
Copy link
Contributor

@muskangulati-qasource could you please retest again to see if this issue is happening? Thanks!

@MadameSheema
Copy link
Member

@muskangulati-qasource can you please validate the fix of this issue on the current BC? Thanks

@MadameSheema MadameSheema added the Theme: rac label obsolete label Aug 25, 2021
@muskangulati-qasource
Copy link
Author

Hi @MadameSheema

We tested this ticket & found that the issue is not yet fixed on the 7.15.0-BC2 environment. Please find below the testing details:

Build Details:

VERSION: 7.15.0-BC2
BUILD: 43742
COMMIT: 113d5f2a745ab9be7d567d13ff906dfd0f5b7a9a
ARTIFACT: https://staging.elastic.co/7.15.0-d7816905/summary-7.15.0.html

Screenshot:
Modal

Thanks!

@MadameSheema
Copy link
Member

@angorayc can you please take a look to the above when you have the chance? thanks :)

@angorayc
Copy link
Contributor

angorayc commented Aug 26, 2021

@muskangulati-qasource I could see that the popover wouldn’t stay while scrolling, but I wasn’t able to toggle popover for multiple rows while one is already open. Do you have a recording of how you reproduced that? Thank you.

Update: This is how I tried to reproduce it, have I missed anything?
overflow_btn

@angorayc
Copy link
Contributor

I'm asking in EUI's channel about the positioning: https://elastic.slack.com/archives/C7QC1JV6F/p1629972678009900

@muskangulati-qasource
Copy link
Author

Hi @angorayc,

We have validated this issue on 8.14.0-BC1 and found that the issue is now fixed as the UI is completely updated for the Alert details flyout🟢

Please find below the testing details:

Build details:

VERSION: 8.14.0
BUILD: 73520
COMMIT: c1513cd7e5a00eab209ba02d30cafd6945d75470

Screen Recording:

Alerts.mp4

Hence, closing this issue and marking it as "Validated".

Thanks!

@muskangulati-qasource muskangulati-qasource added the QA:Validated Issue has been validated by QA label Apr 22, 2024
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:low Addressing this issue will have a low 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 v8.10.0
Projects
None yet
Development

No branches or pull requests

6 participants