Make WordPress Core

Opened 2 months ago

Closed 5 weeks ago

Last modified 5 weeks ago

#62573 closed defect (bug) (fixed)

Icons and thumbnails stick to border after file upload in Media Library

Reported by: sukhendu2002's profile sukhendu2002 Owned by: audrasjb's profile audrasjb
Milestone: 6.8 Priority: normal
Severity: minor Version: 6.7.1
Component: Media Keywords: has-screenshots has-patch has-testing-info
Focuses: ui Cc:

Description

When uploading files via the Media Library in WordPress, the icons or thumbnails of uploaded files appear to be sticking to the border.

Steps to Reproduce:

Go to the WordPress Dashboard.
Navigate to Media > Add New Media File.
Upload one or more files using the drag-and-drop area or the "Select Files" button.
Observe the icons or thumbnails of the uploaded files after the upload is complete.

Attachments (1)

media-library-thumbnail-spacing-issue.png (89.9 KB) - added by sukhendu2002 2 months ago.

Download all attachments as: .zip

Change History (11)

This ticket was mentioned in PR #7894 on WordPress/wordpress-develop by @sukhendu2002.


2 months ago
#1

  • Keywords has-patch added

This PR addresses a UI issue in the Media Library where icons or thumbnails of uploaded files appear to stick to the border of their containers after file upload.

### How to Test:

  1. Navigate to Media > Add New Media File in the WordPress admin dashboard.
  2. Upload a file using the drag-and-drop area or the "Select Files" button.
  3. Verify that the thumbnails/icons of the uploaded files no longer stick to the container's border and are appropriately spaced.

Trac ticket: https://core.trac.wordpress.org/ticket/62573

https://github.com/user-attachments/assets/ca3847f1-0150-44a6-b630-8a6d68a232dc

#2 @im3dabasia1
8 weeks ago

  • Keywords has-testing-info added

Reproduction Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.25
  • Server: nginx/1.27.2
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.25)
  • Browser: Chrome 129.0.0.0
  • OS: macOS
  • Theme: Twenty Twenty-Five 1.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Error condition occurs (reproduced).

Supplemental Artifacts

https://i.postimg.cc/PrJCkcPV/Screenshot-2024-11-27-at-6-54-01-PM.png

Last edited 8 weeks ago by im3dabasia1 (previous) (diff)

#3 @sainathpoojary
8 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://github.com/WordPress/wordpress-develop/pull/7894

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.25
  • Server: nginx/1.27.2
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.25)
  • Browser: Chrome 131.0.0.0
  • OS: macOS
  • Theme: Twenty Twenty-Five 1.0
  • MU Plugins: None activated
  • Plugins: Test Reports 1.2.0

Actual Results

✅ Issue resolved with patch.

Supplemental Artifacts

Before patch:

https://utfs.io/f/PL8E4NiPUWyOl6WlVoUFvL3unmS7e6ZWRODwrzf2jyCcBM9A

After patch:

https://utfs.io/f/PL8E4NiPUWyOphO42EaQVMXblTwtAz0NaoWOrjEIdxin63gS

#4 @im3dabasia1
8 weeks ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://github.com/WordPress/wordpress-develop/pull/7894.diff

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.25
  • Server: nginx/1.27.2
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.25)
  • Browser: Chrome 129.0.0.0
  • OS: macOS
  • Theme: Twenty Twenty-Five 1.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.

Supplemental Artifacts

https://i.postimg.cc/Pr7Pf4DX/Screenshot-2024-11-27-at-6-51-46-PM.png

This ticket was mentioned in Slack in #core-test by oglekler. View the logs.


8 weeks ago

This ticket was mentioned in Slack in #core-media by oglekler. View the logs.


8 weeks ago

#7 @joedolson
8 weeks ago

Introduced in #60141. Pinging @audrasjb.

#8 @audrasjb
8 weeks ago

  • Milestone changed from Awaiting Review to 6.8
  • Owner set to audrasjb
  • Status changed from new to accepted

Thanks for the ping @joedolson! Self assigning for final review.
Let's fix this in 6.8.

#9 @audrasjb
5 weeks ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 59550:

Media: Fix margin issues on the Media file upload screen.

This changeset fixes an issue in the Media Library where icons or thumbnails of uploaded files were stuck to the border of their container after file upload.

Follow-up to [58279].

Props sukhendu2002, im3dabasia1, sainathpoojary, joedolson.
Fixes #62573.
See #60141.

Note: See TracTickets for help on using tickets.