Problem/Motivation

On pressing escape key the entity browser modal popup does not close, While with the core media library as a widget this behaviour is working fine.

Steps to reproduce

  • Enable media module
  • Add a field of type media in article content type
  • Install and enable entity browser module
  • Add an entity browser by navigating to admin/config/content/entity_browser/add as image browser with all default settings
  • Navigate to article >> form display and select entity browser as widget for the media field that you have created
  • Now go to add article form and click a select entities or image a modal popup will open
  • Now press escape key it won't close
Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

arshadkhan35 created an issue. See original summary.

arshadkhan35’s picture

Status: Active » Needs review
StatusFileSize
new642 bytes

Patch for the same.

abhinesh made their first commit to this issue’s fork.

abhinesh’s picture

Version: 8.x-2.5 » 8.x-2.10
Assigned: Unassigned » abhinesh
StatusFileSize
new1.07 KB

I have added the patch re-roll for version 8.x-2.10.

divya.sejekan’s picture

Assigned: abhinesh » Unassigned
StatusFileSize
new1.93 MB

Verified using patch #5 . The issue is fixed and used is able to close the entity browser by esc key.

Testing steps :
- Enable media module
- Add a field of type media in article content type
- Install and enable entity browser module
- Add an entity browser by navigating to admin/config/content/entity_browser/add as image browser with all default settings
- Navigate to article >> form display and select entity browser as widget for the media field that you have created
- Now go to add article form and click a select entities or image a modal popup will open
- Now press escape key it won't close

Keeping the issue in Need review since the MR has some issues

silvi.addweb made their first commit to this issue’s fork.

silvi.addweb’s picture

Status: Needs review » Reviewed & tested by the community

I have tested the patch #5 and it's working fine. also, I have raised MR for the same.

Berdir made their first commit to this issue’s fork.

  • Berdir committed 587ef866 on 8.x-2.x authored by abhinesh
    Issue #3183414 by abhinesh, arshadkhan35: Modal popup does not close on...
berdir’s picture

Status: Reviewed & tested by the community » Fixed

Merged the already existing merge request.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.