#Needs usability review

Updating #174 (2985168-175.patch) to include a hunk from #172 that seems to have been missed or accidentally removed (2 hooks in media_library.module).

The re-added hunk:

• Hides the "Delete" button from the modal
• Makes the modal "Save" button submit via AJAX
• Gets the media-item in the node edit form updated with any changes saved using the modal

Minimally tested, it works well with Drupal core 11.1.7 + Gin theme.

Sorry should of also meant that patches should be in MRs

Looking at #17 will need a test case showing the issue.

Patch in #12 only changes a Drupal core file, while the patch in #2 also changes some of this theme's files. The latter can be used for the merge request.

Hopefully this is good enough.
I've rewritten everything with new approach.
Hopefully no one will be mad, this issue were not going anywhere anyway.

IS still needs to be updated

For instance if your password is mysql://user:pass#pass@host/db it should become `mysql://user:pass%23pass@host/db` to become valid url

Strange this is not fixed even in Drupal 11, and there not a lot of people complaining.

I can confirm this is still present, my password contains '#'. If I leave it like that, url is invalid, and if encode password it is not properly decoded.

You can encode password with urlencode (php function) or just find online url encoder - encode password only.

Attaching patch fix, that will decode url properly. There are a lot of comments telling about this issue on php.net parse_url page.

Also I think I covered #15