Opened 4 months ago
Closed 4 months ago
#32657 closed defect (bug) (fixed)
Media List (wp-list-table): attachment thumbnails overlap each other
Reported by: | tyxla | Owned by: | helen |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | normal | Version: | 4.3 |
Component: | Administration | Keywords: | has-patch |
Focuses: | ui | Cc: |
Description
When viewing the Media Library in list view, the thumbnails overlap the next entry.
Attaching a preview of the issue.
Attachments (4)
Change History (14)
comment:1 @tyxla — 4 months ago
Since the thumbnail container is absolute, I suggest that we fix this by adding a min height of the .media-info. This will work fine for both cases - image thumbnails (which are 60px in height), and icon thumbnails (which are 64px in height).
Patch coming in a minute.
@tyxla — 4 months ago
Adding min height to .media-info to resolve the issue with overlapping thumbnails in media list view.
comment:3 @obenland — 4 months ago
- Milestone changed from Awaiting Review to 4.3
Possibly a result from recent Admin UI work.
/cc @helen @stephdau
comment:4 @stephdau — 4 months ago
Similar/related to https://core.trac.wordpress.org/ticket/32395#comment:10
comment:5 @helen — 4 months ago
Looks like row actions are missing :) Going to dig into that first, though we should probably do this anyway just in case.
comment:7 @helen — 4 months ago
Adding the min-height would make each row taller for not much reason given the default state, though I do wonder if we should protect against this. Thoughts?
comment:8 @tyxla — 4 months ago
Oh well. How could I have missed that the row actions are missing. :) Thanks for noticing that.
But with the latest commit [32798] I have some PHP notices appearing there. Turns out that they are because of bad parameter naming for handle_row_actions(). I've attached a patch that addresses these, and updated the method docs as well.
Also, I've updated the CSS for consistency with that solution. With this approach, min-height will not be necessary anymore.
comment:9 @slackbot — 4 months ago
This ticket was mentioned in Slack in #core by tyxla. View the logs.
comment:10 @helen — 4 months ago
- Owner set to helen
- Resolution set to fixed
- Status changed from new to closed
In 32805:
Attaching a preview of the issue