- π¨π¦Canada teknocat
While there are other related issues that would be ideal to solve in core all with one solution, in the meantime I think the Media Library Edit contrib module is a great solution to this particular problem. It works very well, supports D10 and appears to be fairly actively maintained.
- πͺπΈSpain miguelarber
In a very specific scenario, when a media entity has more than a single 'file' type field (i.e.: 'book' media entity, containing a 'file' field and a 'cover' field) this patch causes the editing form to not submit properly when adding/removing the second 'file' type field (the user has to click on the 'Save' button twice, so that the changes take place).
I have added some extra lines to make sure that the media library opener is actually the 'Field widget'. This avoids the annoying issue I described above. This patch is based on #148.
- last update
almost 2 years ago Custom Commands Failed - last update
almost 2 years ago Custom Commands Failed - πΊπΈUnited States Chris Matthews
I suppose it would not be advisable to get the basic functionality of the Media Library Edit β or the Media Library Media Modify β contrib module in Core until as phenaproxima mentioned in #109 there is ...
dedicated funding and time from the media team, UX team, and accessibility experts to come up with a version of this feature that would be suitable for core.
It definitely would be nice to have a Core solution but the patch in this issue is working for me.
- π§πͺBelgium victonator
As described in #157 I came across the same issue where we had to press the 'Save' button twice.
Patch #157 fixes this and works great for me. - Merge request !4517Issue #2985168: Allow media items to be edited in a modal when using the field widget β (Open) created by dieterholvoet
- last update
over 1 year ago Custom Commands Failed - π§πͺBelgium dieterholvoet Brussels
I opened MR !4517 with #157 β¨ Allow media items to be edited in a modal when using the field widget Postponed rebased on 11.x.
- πͺπΈSpain miguelarber
I am sorry for the delay, I just read the message posted by @Anybody, but I see that @dieterholvoet has already created an MR of the rebased #157. Please let me know if I can help on anything else.
PS: I'm glad to help, thanks for testing @Victornator.
- π«π·France Nicolas S. Lyon, France
patch #153 work with a Drupal 10.1.5
- πΊπΈUnited States byronveale
Updated the patch in comment #157 to work with 10.1, posting here in case anybody else needs itβ¦
- πΊπΈUnited States byronveale
Apologies for the noise but am attaching another patch, #165 with a minor update: added an aria-label to the Edit button, in the same fashion as the one present on the Remove button.
- last update
over 1 year ago Patch Failed to Apply - First commit to issue fork.
- Status changed to Active
over 1 year ago 3:09pm 22 November 2023 - πΊπΈUnited States bkosborne New Jersey, USA
This is getting confusing to work on. There's many patches, and two open merge requests. To keep this sane and easier to review, please let's stick to updating MR !4517 only.
I've updated it to reflect the changes #166 and made another minor fix for a PHP warning.
I also agree with comments above that this shouldn't really be postponed without a related issue that it's waiting on. Until we have that, setting this back to "Active". I acknowledge this issue isn't likely to get anywhere without involvement from the UX team, but having the issue set to Postponed doesn't seem that will make it any more likely to happen.
- Status changed to Postponed
over 1 year ago 3:27pm 22 November 2023 - π¬π§United Kingdom joachim
This is postponed on π Design a UI to allow various kinds of alterations to referenced Media entities in a modal Active which is marked as a parent.
And it should be set to postponed because there's no point people working on this patch until a UI has been designed.
- πΊπΈUnited States bkosborne New Jersey, USA
Thanks @joachim, missed that.
Note that I think the original merge request is what was split off into a module Media Library Media Modify β . This allows overrides of fields from the referenced media. AKA contextual edits. That functionality differs quite a bit from the latest patches/MR from this issue, which now just focus on presenting a simple edit button to edit the media item in a modal. Those changes affect the media everywhere it's used. It's simpler and I think has a much better chance of getting into core.
I hope we can agree to keep this issue focused on the simple use case.
- π¬π§United Kingdom joachim
> That functionality differs quite a bit from the latest patches/MR from this issue, which now just focus on presenting a simple edit button to edit the media item in a modal. Those changes affect the media everywhere it's used. It's simpler and I think has a much better chance of getting into core.
I agree that the override functionality is a bit specific.
But for core we need editability AND translation, which is two items. Overriding is just a third, but the need for a UI pattern is already there.
- π³π±Netherlands arantxio Dordrecht
I created a patch for 10.2 based on the MR @bkosborne updated.
- last update
over 1 year ago 30,637 pass - π³π±Netherlands casey
Snapshot of latest state of MR for safe usage with composer-patches on drupal/core:11.1.1