Skip to content

Comments

[Security Solution] The "Create a New Case Modal" is currently being blocked by the "Top N Popup"#158141

Open
gitstart wants to merge 1 commit intoelastic:mainfrom
gitstart:ELS-109
Open

[Security Solution] The "Create a New Case Modal" is currently being blocked by the "Top N Popup"#158141
gitstart wants to merge 1 commit intoelastic:mainfrom
gitstart:ELS-109

Conversation

@gitstart
Copy link
Contributor

@gitstart gitstart commented May 19, 2023

What does this PR do?

  • The fix implemented in this PR is to modify the code so that the Create a New Case Modal is no longer obscured by the Top N Popup. This will allow users to access the Create a New Case feature seamlessly without any hindrance.

Issue References

Video/Screenshot Demo

FIX:
Screen.Recording.2023-05-19.at.22.04.53.mov

This code was written and reviewed by GitStart Community. Growing great engineers, one PR at a time.

…blocked by the `Top N Popup`

Co-authored-by: Anjola Adeuyi <57623705+anjola-adeuyi@users.noreply.github.com>
Co-authored-by: gitstart_bot <gitstart_bot@users.noreply.github.com>
@kibanamachine
Copy link
Contributor

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@gitstart gitstart changed the title ELS-109 - [Security Solution] The "Create a New Case Modal" is currently being blocked by the "Top N Popup" [Security Solution] The "Create a New Case Modal" is currently being blocked by the "Top N Popup" May 19, 2023
@gitstart gitstart marked this pull request as ready for review May 19, 2023 21:44
@gitstart gitstart requested a review from a team as a code owner May 19, 2023 21:44
@patrykkopycinski
Copy link
Contributor

@gitstart Thank you for setting up a PR, but I believe the issue was already addressed, could you pull the latest main and verify?

@anjola-adeuyi
Copy link
Contributor

@gitstart Thank you for setting up a PR, but I believe the issue was already addressed, could you pull the latest main and verify?

@patrykkopycinski,

Thank you for bringing up this. I have reconfirmed and reverified from the main branch, and it appears that the problem has not been fixed or addressed yet. The only fix that has been implemented so far is for the Inspect Alert Trend Modal, which I mentioned in my previous communication #155424. However, the issue regarding the Create a New Case Modal being blocked still persists on the main branch.

To provide further evidence, I have attached a recent screenshot from the main branch, clearly showing that the problem still exists.

Please let me know if you have any additional questions or if there is anything else I can assist you with. cc: @gitstart

video demonstration

ELS-109.Fix.Explanation.3.mp4

screenshot

Screenshot 2023-05-22 at 10 45 37

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants