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

#1589: Hitting Enter key on Edit Image page redirects the user to Magento Dashboard instead of Saving the image #1597

Conversation

yolouiese
Copy link
Contributor

Need to verify (*)

  • If pressing Enter key closes edit image modal and saves the image details

Description (*)

  • Added event on data-bind attribute and created a function for Enter event key to prevent the page from reloading and to save image details

Fixed Issues (if relevant)

  1. Fixes Hitting "Enter" key on Edit Image page redirects the user to Magento Dashboard instead of Saving the image #1589: Hitting Enter key on Edit Image page redirects the user to Magento Dashboard instead of Saving the image

Manual testing scenarios (*)

  1. Go to Content - Media Gallery
  2. Select an image saved from Adobe Stock and click "three dots" in the bottom right
  3. Select Edit
  4. Place the cursor into Name or Description field
  5. Type any text
  6. Hit Enter key from your keyboard

… to Magento Dashboard instead of Saving the image - prevented page to reload and image details are saved when hitting enter
@yolouiese
Copy link
Contributor Author

@magento run all tests

Copy link
Contributor

@gabrieldagama gabrieldagama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @yolouiese, thanks for the PR, it looks great! Please follow below a comment:

… to Magento Dashboard instead of Saving the image - added key codes library and revised key code condition
@yolouiese
Copy link
Contributor Author

@magento run all tests

gabrieldagama
gabrieldagama previously approved these changes Jul 16, 2020
… to Magento Dashboard instead of Saving the image - fixed the error in static test invalid case of type of booelean
@gabrieldagama
Copy link
Contributor

@magento run all tests

1 similar comment
@yolouiese
Copy link
Contributor Author

@magento run all tests

Copy link
Contributor

@gabrieldagama gabrieldagama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, the failing tests are not related.

@chalov-anton
Copy link
Contributor

✔️ QA PAssed

Peek 2020-07-17 12-37

@ghost
Copy link

ghost commented Jul 17, 2020

Hi @yolouiese, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hitting "Enter" key on Edit Image page redirects the user to Magento Dashboard instead of Saving the image
3 participants