Make WordPress Core

Opened 9 months ago

Closed 9 months ago

#64201 closed defect (bug) (fixed)

Remove comments column for Media list view

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

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 9 months ago.

Download all attachments as: .zip

Change History (11)

#1 follow-up: @westonruter
9 months ago

  • Description modified (diff)
  • Keywords needs-patch has-screenshots added
  • Milestone Awaiting ReviewFuture 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.


9 months 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
9 months ago

Replying to westonruter:

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

Correct.

#4 @westonruter
9 months ago

  • Milestone Future Release7.0

#5 @westonruter
9 months ago

  • Owner set to westonruter
  • Status newreviewing

@westonruter commented on PR #10471:


9 months ago
#6

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

#7 @soyebsalar01
9 months ago

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

Follow the below Image :

https://ibb.co/1G358nVC

#8 @westonruter
9 months ago

@soyebsalar01 what is the value of your wp_attachment_pages_enabled option?

@shailu25 commented on PR #10471:


9 months 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
9 months ago

  • Resolutionfixed
  • Status reviewingclosed

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.