Make WordPress Core

Opened 5 weeks ago

Closed 3 weeks ago

#64201 closed defect (bug) (fixed)

Remove comments column for Media list view

Reported by: knutsp's profile knutsp Owned by: westonruter's profile westonruter
Milestone: 7.0 Priority: normal
Severity: trivial Version: 6.4
Component: Media Keywords: has-screenshots has-patch
Focuses: ui, administration Cc:

Description (last modified by westonruter)

In case empty( get_option('wp_attachment_pages_enabled') ) remove the then unnecessary 'comments' column.

This is a follow-up to #57913.

Attachments (1)

media-library-list-view-with-comments.png (296.6 KB) - added by westonruter 5 weeks ago.

Download all attachments as: .zip

Change History (11)

#1 follow-up: @westonruter
5 weeks ago

  • Description modified (diff)
  • Keywords needs-patch has-screenshots added
  • Milestone changed from Awaiting Review to Future Release

@knutsp The screenshot media-library-list-view-with-comments.png shows what you're referring to, correct?

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


5 weeks ago
#2

  • Keywords has-patch added; needs-patch removed

PR for Trac ticket: https://core.trac.wordpress.org/ticket/64201

This PR removes the “Comments” column from the Media Library when attachment pages are disabled.

When the wp_attachment_pages_enabled option is empty or evaluates to false, attachments no longer have dedicated front-end pages. In such cases, displaying the “Comments” column in the Media Library becomes redundant, since comments are not accessible or visible on attachment pages.

#### Benefits:

  • Prevents confusion for users by hiding irrelevant UI
  • Keeps the Media Library interface cleaner
  • Aligns with the expected behaviour when attachment pages are disabled

#3 in reply to: ↑ 1 @knutsp
5 weeks ago

Replying to westonruter:

@knutsp The screenshot media-library-list-view-with-comments.png shows what you're referring to, correct?

Correct.

#4 @westonruter
4 weeks ago

  • Milestone changed from Future Release to 7.0

#5 @westonruter
4 weeks ago

  • Owner set to westonruter
  • Status changed from new to reviewing

@westonruter commented on PR #10471:


4 weeks ago
#6

I'll commit this after the 6.9 branch is created for the RC1 on Tuesday.

#7 @soyebsalar01
4 weeks ago

I tested locally and found comments columns and the issue is still persist

Follow the below Image :

https://ibb.co/1G358nVC

#8 @westonruter
4 weeks ago

@soyebsalar01 what is the value of your wp_attachment_pages_enabled option?

@shailu25 commented on PR #10471:


4 weeks ago
#9

BeforeAfter
https://github.com/user-attachments/assets/d2214ac3-ad39-4429-a532-8e3ae86bc488 https://github.com/user-attachments/assets/511164f2-a628-4e06-802f-8e2010d18f28

#10 @westonruter
3 weeks ago

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

In 61268:

Media: Remove Comments column from Media list view when attachment pages are not enabled.

Developed in https://github.com/WordPress/wordpress-develop/pull/10471

Follow-up to [56657].

Props akshat2802, knutsp, shailu25, soyebsalar01, westonruter.
See #57913.
Fixes #64201.

Note: See TracTickets for help on using tickets.