Modal auto-open conflicts with AJAX

If return statement in latest patch causes NS_BINDING_ABORTED breaking referencing of existing entities in versions >2.6.

This seems to be working working better for what ever reason.

#9 seems to work wonderfully against E.B. 2.9.0 and Drupal 9.4.x.

Wrapping things in this pseudo-code seems to be an elegant solution, but I do think that using drupal/once would be more elegant. IMO it would be worth merging this bug fix in as-is and opening a follow-up task to refactor things to drop jquery / use drupal/once.

TL;DR - +1 RTBC

If the focus is on another field (for example a text field) before opening the entity browser modal & closing it by submitting its form, the page is scrolled to the previously focused page element. To prevent it, we can add .focus() before triggering the .click().

Hi,
Patch from #10 is fixing this in my case.

jan kellermann → made their first commit to this issue’s fork.

Merge Request works for me.

@jan kellermann thanks for the MR - would you confirm RTBC? Then I'd merge it.

Okay let's merge this and create a follow-up to use core/once here in the future.