[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

elastic / kibana

Your window into the Elastic Stack

https://www.elastic.co/products/kibana

Other

19.72k stars 8.13k forks source link

[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 closed 5 months ago

muskangulati-qasource commented 3 years ago

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:

Kibana user should be logged in.
Alert should be generated

Steps to Reproduce:

Navigate to the alerts tab.
Click on '>' icon
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

elasticmachine commented 3 years ago

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

manishgupta-qasource commented 3 years ago

Reviewed & assigned to @MadameSheema

elasticmachine commented 3 years ago

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

MadameSheema commented 3 years ago

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

angorayc commented 3 years ago

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

MadameSheema commented 3 years ago

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

muskangulati-qasource commented 3 years ago

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 commented 3 years ago

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

angorayc commented 3 years ago

@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 commented 3 years ago

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

muskangulati-qasource commented 5 months ago

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:

https://github.com/elastic/kibana/assets/60252716/350f7356-6a0f-40c5-a8ac-7274ea468cd0

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

Thanks!