Save button missing in modal for editing media using Media Library Edit module

I have the same issue with the combination of Gin and Media Library Edit, since the upgrade to 3.11

Changing the Twig template also fixes it for me. Does this have anything to do with the new sticky buttons functionality destined for 4.x?

Same here!

This regressions started with Gin release 8.x-3.0-rc11.

Can confirm Gin is not rendering the bottom of the edit modal, more specifically the 'ui-dialog-buttonpane' at the bottom. I'm using the media_library_media_modify module

My merge requests assumes that Gin content forms should never be used for Ajax requests, because forms in Ajax requests don't render the page template or the local actions block. This fixes it for the media library edit button, and theoretically this should also prevent similar problems with modules like Inline Entity Forms.

My assumption is a fairly broad approch, so feedback is welcome.

Thank you @hundri! I've verified your fix and it works very well!

@hundri thanks for the fix, it works well for me as well.

@hudri thank you for the MR 🙇

I left you a question 👀

@hudri sounds good 👍

Waiting for another round of re-reviews to confirm it's still working properly, afterwards we can merge this ASAP and issue a new release 👀

Tests from Drupal commerce users would be helpful, this would cross-check compatiblity with Inline Entity Form module.

I didn't check with Inline Entity Form, but just in case I tested compatibility with other modules, where it could potentially impact behaviour:
- Views Bulk Operations
- Views Entity Form Field
- Dashboards with Layout Builder
All of them seems to be working fine, with or without this patch :)

Here is the patch for people wanting to apply it directly on RC11.
I can confirm it fixes the issue for the media edit modal introduced by the Core patch here ✨ Allow media items to be edited in a modal when using the field widget Postponed

@DuaelFr, there are small differences between your patch and what is in mr. Can you please clarify them? I'm wondering what is the goaald and which one should be used for patching RC11 on my website

@besek MR's patch is against 3.x-dev so it does not apply on rc11. This patch is a workaround for people who need this change but cannot wait for the next release and/or cannot switch to the development version of the theme.

Ok, thanks for clarification and the patch. I was testing mr on rc11 and it worked fine, so I thought this would be enough to use it as a patch.

We applied patch from #18 and media library edit modal now diaplays save button.
Tried it on Drupal 10.3, gin 3.x-rc11 and media_library_edit 3.0.3

Thanks! I've also ported this to the 4.0.x branch

Thank you very much for fixing this! :) I was looking for the reason and finally found it here.
Is a new -rc12 release planned soon? Looks like several issues were recently fixed in -dev. Or should we use the patch?

This was released today

Thank you very very much @saschaeggi! :)

Thank you @saschaeggi

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

Sorry to reopen this, but it seems that it does not fix the problem for the Entity Browser Modal

And the patch :)

I'm seeing the same issue in views - cannot change settings unless I right click, open in new tab (ie no modal)

I'm also experiencing this issue in version rc13 (D10.3.1). Using the media_library_media_modify module. It seems to work with media fields in paragraphs but not in media fields directly in nodes. Patch in #29 fixed it and works for me.

@saschaeggi then *ping* for reopening?

No need to reopen this, see the related open issues/MRs which need a review instead.

Sorry didn't see that =/