[wp-trac] [WordPress Trac] #55393: File name URL overflow the container in the media editor

WordPress Trac noreply at wordpress.org
Tue Mar 15 23:37:37 UTC 2022


#55393: File name URL overflow the container in the media editor
--------------------------+--------------------------------------
 Reporter:  mitogh        |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  6.0
Component:  Media         |     Version:  5.4
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:  ui, css, administration
--------------------------+--------------------------------------
Changes (by costdev):

 * focuses:  ui, administration => ui, css, administration
 * version:  5.9.2 => 5.4
 * milestone:  Awaiting Review => 6.0


Comment:

 == Issue Reproduction Report

 === Env
 * Web Server: Apache (Linux)
 * WordPress: 5.9.2
 * Browser: Firefox 98.0.1
 * OS: Windows 10
 * Theme: TT1 Blocks
 * Plugins: None activated.

 === Steps to reproduce
 1. Navigate to `Media > Add New`.
 2. Upload a large image with a long filename.
 3. Navigate to `Media > Library`.
 4. Click the "List Mode" icon.
 5. Click the "Edit" link for the image you uploaded.

 To the right, the "Original image:" link shuold overflow the container.

 === Results
 1. Issue reproduced in Firefox. ✅
 2. Could not reproduce in Chrome (incl. Edge, Brave). ❌

 === Notes
 1. Introduced in [https://core.trac.wordpress.org/changeset/47202 47202].
 2. `word-break: break-word` is used on the bolded "File name:" text above
 the link. This also works when applied to the "Original image:" link,
 however I personally think that `word-break: break-all` looks cleaner.
 3. Adding `css` focus for input on preference between `break-word` and
 `break-all`.
 4. [https://core.trac.wordpress.org/changeset/47202 47202] introduced this
 link in the three locations below. The patch should be tested all three of
 these.
   - [https://core.trac.wordpress.org/browser/trunk/src/wp-includes/media-
 template.php#L442 wp-includes/media-template.php:442].
   - [https://core.trac.wordpress.org/browser/trunk/src/wp-includes/media-
 template.php#L665 wp-includes/media-template.php:665].
   - [https://core.trac.wordpress.org/browser/trunk/src/wp-
 admin/includes/media.php#L3485 wp-admin/includes/media.php:3485] - This is
 the one reported in this ticket.
 5. Milestoning for `6.0`.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/55393#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list