WordPress.org

Make WordPress Core

#45028 closed defect (bug) (fixed)

Comments bulk actions aren't displaying below the comments list

Reported by: jobthomas Owned by: melchoyce
Milestone: 5.1 Priority: normal
Severity: normal Version:
Component: Comments Keywords: has-screenshots
Focuses: administration Cc:
PR Number:

Description

On mobile, there are no bulk actions on comments, contrary to pages and posts. This also results in a weird "item" overlap with the search box. See screenshots.

Attachments (4)

pages.png (64.4 KB) - added by jobthomas 14 months ago.
comments.png (51.2 KB) - added by jobthomas 14 months ago.
45028.diff (797 bytes) - added by shaneeckert 14 months ago.
45028 Fix where extra items may be empty, which will prevent the table nav from displaying later.
45028.2.diff (1.3 KB) - added by dd32 14 months ago.
Code formatting changes required to 45028.diff

Download all attachments as: .zip

Change History (17)

@jobthomas
14 months ago

@jobthomas
14 months ago

#1 @jobthomas
14 months ago

  • Component changed from General to Administration
  • Focuses administration added

#2 @jobthomas
14 months ago

  • Summary changed from On mobile, comments bulk actions aren't displaying to Comments bulk actions aren't displaying below the comments list

#3 @mukesh27
14 months ago

  • Component changed from Administration to Comments
  • Keywords needs-patch ui-feedback has-screenshots dev-feedback added

@shaneeckert
14 months ago

45028 Fix where extra items may be empty, which will prevent the table nav from displaying later.

#4 @dd32
14 months ago

  • Keywords commit added

After going down the rabbit hole of what $extra_items is, 45028.diff looks good to me.

#5 @melchoyce
14 months ago

  • Owner set to melchoyce
  • Resolution set to fixed
  • Status changed from new to closed

In 43671:

List Table: Restore the List Table items property to allow Bulk Actions to be properly determined.

This fixes the display of Bulk Actions where there exists only a single page of results.

Props shaneeckert, jobthomas, dd32.
Fixes #45028.

@dd32
14 months ago

Code formatting changes required to 45028.diff

#6 @melchoyce
14 months ago

In 43672:

List Table: Fix formatting issue introduced in r43671.

Props dd32.
See #45028.

#7 @netweb
13 months ago

  • Milestone changed from Awaiting Review to 5.0

#8 @johnbillion
13 months ago

  • Keywords needs-patch ui-feedback dev-feedback commit removed
  • Milestone changed from 5.0 to 5.0.1
  • Resolution fixed deleted
  • Status changed from closed to reopened

#9 @pento
11 months ago

  • Milestone changed from 5.0.1 to 5.0.2

#10 @pento
11 months ago

  • Milestone changed from 5.0.2 to 5.0.3

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


10 months ago

#12 @desrosj
10 months ago

  • Keywords fixed-major added

#13 @desrosj
10 months ago

  • Keywords fixed-major removed
  • Milestone changed from 5.0.3 to 5.1
  • Resolution set to fixed
  • Status changed from reopened to closed

Let's leave this in 5.1 as this is outside of the 5.0.3 scope.

Note: See TracTickets for help on using tickets.