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

Fix Loader stays over the media gallery if it was opened second time from different control on the same page #1163

Closed

Conversation

sidolov
Copy link
Contributor

@sidolov sidolov commented Apr 7, 2020

Description (*)

Depends on

magento/magento2#27566

Related Pull Requests

magento/magento2#27566
magento/magento2-page-builder#449

PB PR

magento/magento2-page-builder#449

Fixed Issues (if relevant)

  1. Loader stays over the media gallery if it was opened second time from different control on the same page #935: Loader stays over the media gallery if it was opened second time from different control on the same page
  2. ...

Manual testing scenarios (*)

Preconditions (*)

  1. Install Magento and Adobe Stock Integration See documentation

Steps to reproduce (*)

  1. Login to Admin panel
  2. Open Media Gallery (i.e. Catalog -> Category -> expand Content -> Select from Gallery button)
  3. Close the Media Gallery
  4. Open the Media Gallery using different entry point on the same page (i.e. WYSIWYG)

Expected result (*)
Media Gallery is loaded and loader disappears

@Nazar65 Nazar65 changed the title magento/magento2#27566: Do not re-load the gallery app, just content Fix Loader stays over the media gallery if it was opened second time from different control on the same page Apr 9, 2020
@Nazar65
Copy link
Member

Nazar65 commented Apr 16, 2020

Closing this one as changes mergend into #1186

@Nazar65 Nazar65 closed this Apr 16, 2020
@ghost
Copy link

ghost commented Apr 16, 2020

Hi @sidolov, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants