WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#45028 closed defect (bug) (fixed)

Comments bulk actions aren't displaying below the comments list

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

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 6 years ago.
comments.png​ (51.2 KB) - added by jobthomas 6 years ago.
45028.diff​ (797 bytes) - added by shaneeckert 6 years 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 6 years ago.
Code formatting changes required to 45028.diff

Download all attachments as: .zip

Change History (17)

@jobthomas
6 years ago

@jobthomas
6 years ago

#1 @jobthomas
6 years ago

  • Component changed from General to Administration
  • Focuses administration added

#2 @jobthomas
6 years 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
6 years ago

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

@shaneeckert
6 years ago

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

#4 @dd32
6 years ago

  • Keywords commit added

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

#5 @melchoyce
6 years 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
6 years ago

Code formatting changes required to 45028.diff

#6 @melchoyce
6 years ago

In 43672:

List Table: Fix formatting issue introduced in r43671.

Props dd32.
See #45028.

#7 @netweb
6 years ago

  • Milestone changed from Awaiting Review to 5.0

#8 @johnbillion
6 years 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
6 years ago

  • Milestone changed from 5.0.1 to 5.0.2

#10 @pento
6 years ago

  • Milestone changed from 5.0.2 to 5.0.3

This ticket was mentioned in ​Slack in #core by audrasjb. ​View the logs.
6 years ago

#12 @desrosj
6 years ago

  • Keywords fixed-major added

#13 @desrosj
6 years 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.