Media library browser spacing issues

I have fixed the spacing issue and created patch. Attached screenshot for reference.

Also found one more issue modal box close icon alignment issue. Added fixes in this patch.

I have updated patch

Thanks for the patches but I'd say patching the media_library module shouldn't be the proper approach, as the module itself doesn't have the issue with different admin themes like Seven or Claro.

Either way thanks for adding the RTL fix too, that was a really good spot.

dabodia I have verified your comment. As you mentioned above i'm not using from media_library module. Instead i'm using web/themes/contrib/gin/styles/components/media_library.scss.

@Vinayak.Ambig, I checked the patch mentioned in #5, and it appears it hasn't been applied. I verified this on both Drupal 10 and Drupal 9.

Can you share screenshots where you are getting spacing issues? I tested on Drupal 10 and I did not find anything misaligned.

It appears that the problem has already been addressed. I attempted to reproduce the issue, but I discovered that the required CSS code has already been implemented.

.media-library-wrapper {
    margin-top: 0;
    margin-left: calc(var(--gin-spacing-m) * -1);
    margin-right: calc(var(--gin-spacing-m) * -1);
}<code>

Hi,
I have tested on the Drupal 10. 2 and Drupal 9.5 versions.

Testing steps:

Install Drupal version
Install the Gin theme and gin toolbar
Install media and Media Library core module
Add a Midea field for any content type with multiple options
Create a page for content type, and add media
Check for the Add media modal

Test Result:
The cross icon is aligned correctly, but the 'Apply Filter' button at the bottom is slightly misaligned for Drupal 10.2 and for Drupal 9.5 all looks good

Attaching screenshots for reference.

Hey 👋 I've just tried to reproduce this issue but I cannot see any weird alignments here. Nothing visually bad with the close icon of the modal nor the "Apply filters" button.

The only wrong thing I can see here are the vertical tabs that you can see when having multiple media bundles enabled for referencing.

I also tried to reproduce the issue with given steps. I was not able to find any spacing or alignment issues and everything looked good to me.

I just upgraded drupal core from 10.1 to 10.2,and the media library browser displayed not correctly like #20, the media type filter style lost.

@shweta__sharma please create a new issues for that, this this issue for something else.

For people having issues after upgrading to Drupal 10.2 with vertical tabs having lost their styles: 🐛 [Drupal 10.2 regression] Media Library "widget" View media type tabs have lost styling RTBC