-
Notifications
You must be signed in to change notification settings - Fork 91
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
Pressing escape key confirms the save/license confirmation popup #572
Comments
Hello @sivaschenko , |
@konarshankar07 can you reproduce the same issue with any othe prompts in Magento Admin Panel? |
Hi @konarshankar07 would you like to continue working on this issue and identify the backward compatible fix as the intial fix was not accepted to magento2? |
Hello @sivaschenko .. |
Hello @sivaschenko ... |
Hi @konarshankar07 I believe it was fixed by you in magento/magento2#25349 However, I am keeping this issue open due to comments in the pull request |
Hello @sivaschenko ... |
Hi @konarshankar07 can you please add a link to commit or pull request of the latest fix for future reference |
Preconditions
Steps to reproduce
Actual result
Image preview is saved
Expected result
Popup should be closed and image should not be saved
The text was updated successfully, but these errors were encountered: